TopBookmarksView loads in a flash

RESOLVED FIXED in Firefox 26

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: sriram, Assigned: sriram)

Tracking

unspecified
Firefox 26
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-fig)

Attachments

(1 attachment)

There is a blank screen.
BOOM!
The thumbnails flash on the screen.

Not so good.

There is a blank screen.
The thumbnails zoom-in one by one.

Yaaay!
(Assignee)

Updated

6 years ago
Flags: needinfo?(ibarlow)
(Assignee)

Comment 1

6 years ago
I'm open for other suggestions too. But let it not be bland please :D
(Assignee)

Updated

6 years ago
Assignee: nobody → sriram
(Assignee)

Comment 2

6 years ago
Posted patch PatchSplinter Review
This re-uses grow-fade-in-from-center animation for each thumbnail. Each one appears one after another. This can be changed. We can specify a delay for the rows and the columns.
(Assignee)

Comment 3

6 years ago
Comment on attachment 785143 [details] [diff] [review]
Patch

UX is happy with this animation.
Attachment #785143 - Attachment description: WIP → Patch
Attachment #785143 - Flags: review?(mark.finkle)
Attachment #785143 - Flags: review?(mark.finkle) → review+
(In reply to Sriram Ramasubramanian [:sriram] from comment #2)
> Created attachment 785143 [details] [diff] [review]
> Patch
> 
> This re-uses grow-fade-in-from-center animation for each thumbnail. Each one
> appears one after another. This can be changed. We can specify a delay for
> the rows and the columns.

Looks nice. We may want to refine a bit in followup bugs but this is a great improvement over what we have now.
Flags: needinfo?(ibarlow)
https://hg.mozilla.org/mozilla-central/rev/9040f8043f46
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
You need to log in before you can comment on or make changes to this bug.