Closed
Bug 726875
Opened 13 years ago
Closed 13 years ago
Tabs-Tray doesn't show selected tab when it is displayed
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 13
People
(Reporter: sriram, Assigned: sriram)
Details
Attachments
(1 file)
2.01 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
With more than 7 tabs, on Galaxy Nexus, the list doesn't scroll to show the selected tab. It stops mid-way -- like around 4 or so.
Assignee | ||
Comment 1•13 years ago
|
||
I initially felt that onWindowChanged() is called before list populates that causes this issue. Somehow the "smooth scrolling" fails, even if I move to just after requestLayout() of the container.
This patch ensures that the selected tab is shown when TabsTray displayed. This also takes care of Eclair. However, we lose the "smooth scrolling" to selected tab.
I somehow feel that there is too much animation happening -- a fade-in + drop down animation for the tray along with scrolling of the list. Without the scrolling the tabs-tray has just one animation and it doesn't seem that complex. (A better way of saying I don't have a patch for it yet ;) )
Attachment #596873 -
Flags: review?(mark.finkle)
Updated•13 years ago
|
Attachment #596873 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 2•13 years ago
|
||
Comment 3•13 years ago
|
||
Assignee: nobody → sriram
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 13
Comment 4•13 years ago
|
||
Nightly 13.0a1 (2012-02-16)
Device: Samsung Nexus S - Android 2.3.6
Verified with 8-12 tabs. Selected tab was always displayed in tab menu.
Status: RESOLVED → VERIFIED
Updated•5 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•