Closed Bug 701994 Opened 13 years ago Closed 13 years ago

Tab counter animation should show overlapping of numbers

Categories

(Firefox for Android Graveyard :: General, defect, P3)

All
Android
defect

Tracking

(firefox11 fixed, fennec11+)

VERIFIED FIXED
Tracking Status
firefox11 --- fixed
fennec 11+ ---

People

(Reporter: sriram, Assigned: sriram)

Details

Attachments

(1 file, 1 obsolete file)

As per the UI spec, the tab counter animation should show overlapping of numbers.
Attached patch Patch (obsolete) — Splinter Review
I tried re-implementing TextSwitcher as a Odometer. This moves all the logic to a new file and does the overlap.
The magic was in setting the amount to displace (currently 30%) :sigh: :( :(
Assignee: nobody → sriram
Attachment #574013 - Flags: review?(mark.finkle)
Attached patch PatchSplinter Review
If "the amount to displace" was the "magic" for showing a overlap, I was wondering if that would work with TextSwitcher also.

Strangely it does!! :(
After 2 hours of working on the previous patch, I figured out it takes just 4 line change in existing code to achieve the same effect! :big-sigh: :(
Attachment #574014 - Flags: review?(mark.finkle)
OS: Mac OS X → Android
Hardware: x86 → All
Attachment #574013 - Attachment is obsolete: true
Attachment #574013 - Flags: review?(mark.finkle)
Attachment #574014 - Flags: review?(mark.finkle) → review+
Priority: -- → P3
https://hg.mozilla.org/projects/birch/rev/859ecdfe0168
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Neat.

Sririam, did you file a bug for tweaking the animation speed?

20111115040200
http://hg.mozilla.org/projects/birch/rev/97283e503e1e
Status: RESOLVED → VERIFIED
tracking-fennec: --- → 11+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: