Closed
Bug 748161
Opened 12 years ago
Closed 12 years ago
Improve visible tab indication in tab list menu
Categories
(Firefox :: Theme, defect)
Firefox
Theme
Tracking
()
RESOLVED
FIXED
Firefox 15
People
(Reporter: smaug, Assigned: fryn)
References
Details
Attachments
(3 files)
If one has enough tabs to get the down arrow to open tab list, the styling of those items in the list which are hidden in the tab bar have strange white-ish styling on linux (Fedora 15, Gnome 3, default theme). Looks very unpolished and nothing like anything else in the OS.
Reporter | ||
Updated•12 years ago
|
tracking-firefox14:
--- → ?
Updated•12 years ago
|
Assignee | ||
Comment 1•12 years ago
|
||
(In reply to Olli Pettay [:smaug] from comment #0) > If one has enough tabs to get the down arrow to open tab list, the styling > of those > items in the list which are hidden in the tab bar have strange white-ish > styling on > linux It's the visible tabs that have the modified styling. The hidden tabs have the native styling, AIUI.
OS: Linux → All
Hardware: x86_64 → All
Summary: The hidden tabs have strange styling in the "Tab list" dropdown → Improve visible tab indication in tab list menu
Version: unspecified → Trunk
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #617994 -
Flags: ui-review?(shorlander)
Comment 4•12 years ago
|
||
Comment on attachment 617993 [details] [diff] [review] patch DarkGray may be too close to the native menu color with some Linux and Windows themes. Can we use ThreeDShadow or something like that?
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Dão Gottwald [:dao] from comment #4) > Comment on attachment 617993 [details] [diff] [review] > patch > > DarkGray may be too close to the native menu color with some Linux and > Windows themes. Can we use ThreeDShadow or something like that? Absolutely. I just tried it, and it works fine. Could you review the patch assuming that I replace DarkGray with ThreeDShadow?
Reporter | ||
Comment 6•12 years ago
|
||
(In reply to Frank Yan (:fryn) from comment #1) > It's the visible tabs that have the modified styling. > The hidden tabs have the native styling, AIUI. Ah, that is possible. Anyway the current setup is visually quite disturbing, and something that I don't see elsewhere in Gnome, nor in Firefox itself.
Assignee | ||
Comment 7•12 years ago
|
||
(In reply to Olli Pettay [:smaug] from comment #6) > Anyway the current setup is visually quite disturbing, > and something that > I don't see elsewhere in Gnome, nor in Firefox itself. It's definitely not pretty, but we didn't think of a better way to indicate it at the time given the restrictions of menuitem styling. I don't think it has to match something else in the product. That's not a requirement at all and is only something that gets brought up when people don't like how something looks. I've thought of something prettier and less jarring now, and I wrote a patch for it above.
Comment 8•12 years ago
|
||
Comment on attachment 617993 [details] [diff] [review] patch r=me with ThreeDShadow instead of DarkGray
Attachment #617993 -
Flags: review?(dao) → review+
Assignee | ||
Comment 9•12 years ago
|
||
Comment on attachment 617994 [details]
screenshot of patch
Got ui-review+ from shorlander in-person.
This solution isn't great, but it's better than what we have now, and the metaphor of a scrollbar-like bar representing the current position might come across.
Attachment #617994 -
Flags: ui-review?(shorlander)
Assignee | ||
Comment 10•12 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/313ba4818662
Whiteboard: [fixed-in-fx-team]
Comment 11•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/313ba4818662
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 15
Comment 12•12 years ago
|
||
This change has now caused the over-flow arrow to appear when its not needed, pressing the over-flow arrow only scrolls in a 'blank' glass area on Win7 in the tab-strip. This with 8 tabs open, where before the same number of tabs did not trigger the tab-strip over-flow arrow.
Comment 13•12 years ago
|
||
Ewww, another side affect I just noticed. Refreshing the page causes the tab strip to drop the over-flow arrow, then once page is loaded, the tab-strip shrinks tab-width to redisplay the over-flow arrow. Visually annoying.
Comment 14•12 years ago
|
||
(In reply to Jim Jeffery not reading bug-mail 1/2/11 from comment #13) > Ewww, another side affect I just noticed. Refreshing the page causes the > tab strip to drop the over-flow arrow, then once page is loaded, the > tab-strip shrinks tab-width to redisplay the over-flow arrow. Visually > annoying. Hm, sorry for spam, only see that happening after posting a comment to the bug and hitting submit, other pages that I refresh do not cause the tab strip to change.
Comment 15•12 years ago
|
||
Never-mind my above comments. went back to an older hourly and the tab strip is still behaving strangely.
Comment 16•12 years ago
|
||
What do we believe this is a regression from? Bug 626903?
Assignee | ||
Comment 17•12 years ago
|
||
Reason for removing tracking flag: It's not a functionality regression. The complaint is just that the indication made the menu uglier, which is a valid complaint. I made it prettier now, but there's no hurry to get this onto aurora just for Firefox 14. It's a visual fix for secondary UI, and no functionality is really being affected. To use a cliché: we have bigger fish to fry.
tracking-firefox14:
? → ---
Reporter | ||
Comment 18•12 years ago
|
||
Well, UI is what users see. Bugs there are as bad as bugs in the functionality. (Btw, on trunk the styling looks good. All the items in the list have the same styling.)
Comment 19•12 years ago
|
||
The 'gray-stripe' down the right side looks bad. Is that supposed to be there ? It looks like a psuedo-slider of sorts, but we know the list is limited so it can't be a slider.
Comment 20•12 years ago
|
||
This patch was in a range which caused a Ts regression, so I backed out the whole range: https://hg.mozilla.org/mozilla-central/rev/24a6a53c714a Please reland after investigating and fixing the regression.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 21•12 years ago
|
||
Relanded: https://hg.mozilla.org/integration/fx-team/rev/e2d386af93fa
Status: REOPENED → ASSIGNED
Whiteboard: [fixed-in-fx-team]
Comment 22•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e2d386af93fa
Status: ASSIGNED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Comment 23•12 years ago
|
||
(In reply to Jim Jeffery not reading bug-mail 1/2/11 from comment #19) > Created attachment 619896 [details] > Gray stripe down right side > > The 'gray-stripe' down the right side looks bad. Is that supposed to be > there ? > It looks like a psuedo-slider of sorts, but we know the list is limited so > it can't be a slider. The gray stripe is the indicator that those are the tabs that are visible in the tab strip. The tabs outside of the gray stripe are the ones that are not in the tab strip view due to the overflow mode.
You need to log in
before you can comment on or make changes to this bug.
Description
•