Closed Bug 901066 Opened 9 years ago Closed 8 years ago

TopBookmarksView loads in a flash

Categories

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: sriram, Assigned: sriram)

References

Details

(Whiteboard: fixed-fig)

Attachments

(1 file)

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!
Flags: needinfo?(ibarlow)
I'm open for other suggestions too. But let it not be bland please :D
Assignee: nobody → sriram
Attached 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.
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
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.