Closed
Bug 348975
Opened 18 years ago
Closed 18 years ago
Tab strip color does not match OS theme
Categories
(Firefox :: Tabbed Browser, defect)
Tracking
()
RESOLVED
FIXED
Firefox 2
People
(Reporter: dao, Assigned: mconnor)
References
Details
(Keywords: fixed1.8.1, Whiteboard: [Fx2 theme change])
Attachments
(4 files)
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060816 Minefield/3.0a1
Build Identifier:
Looks like the new Winstripe tab strip goes together with Luna only. I suggest to use semi-transparent pngs to overlay the native color.
Reproducible: Always
Reporter | ||
Updated•18 years ago
|
Flags: blocking-firefox2?
Version: unspecified → 2.0 Branch
Reporter | ||
Comment 1•18 years ago
|
||
Reporter | ||
Updated•18 years ago
|
Reporter | ||
Comment 2•18 years ago
|
||
Other examples where it looks odd:
http://img81.imageshack.us/img81/4144/crappygobuttoncu0.png
http://img53.imageshack.us/img53/4818/alltabod8.jpg
Comment 3•18 years ago
|
||
I agree.
While the effect of 'Depth' on the Tabstrip looks good, the color itself should be derived from the native background color of the theme/OS.
Comment 4•18 years ago
|
||
Neither of the problems in comment 2 seem like this bug.
The first issue is, in theory, fixed by the checkin for bug 347416 (haven't tested that).
The second issue I haven't seen a bug for; it should probably be filed as a separate bug.
Reporter | ||
Comment 5•18 years ago
|
||
(In reply to comment #4)
> Neither of the problems in comment 2 seem like this bug.
There are other problems, too. Nevertheless, the tab strip color doesn't match the OS.
Comment 6•18 years ago
|
||
The new tabbox color makes a mismatch with the beige color of my toolbars.
It is color 240 240 230 in the middle and it changes gradually into color 239 236 217 on the right side.
Comment 7•18 years ago
|
||
This is a screenshot showing the hard-coded tab strip color on Classic. I think there's a problem with the tabs too, are they supposed to be white, or should they match the theme more?
Comment 8•18 years ago
|
||
I think it's safe to say this can be marked as confirmed. Can someone do so please? I don't have the privs as of yet. Thank you
Also the double horizontal lines under the tab and behind each tab look wrong. Is that intentional? The bottom horizontal line doesn't match OS theme color either.
~B
Comment 10•18 years ago
|
||
I don't believe anything in this new tab bar is os theme native.
and whoops... been forgetting the user agent
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060817 BonEcho/2.0b1 ID:2006081703
Updated•18 years ago
|
Flags: blocking-firefox2? → blocking-firefox2+
Target Milestone: --- → Firefox 2
Updated•18 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 11•18 years ago
|
||
Here is an edit of the tab strip background to use semi-transparency instead of the Luna background color.
Reporter | ||
Comment 12•18 years ago
|
||
What has to be taken into account, too, is that WinXP themes tend to use gradients for toolbars:
http://img46.imageshack.us/img46/18/sshot1cf0.jpg
Hence it's not simply a native color + non-native semi-transparent image. Now I wonder, is there a container (with the native appearance) that can be *overlayed* (e.g. with attachment 234310 [details]) or does the new tab strip actually *replace* the native background?
Assignee | ||
Updated•18 years ago
|
Whiteboard: [Fx2 theme change]
Updated•18 years ago
|
Assignee: nobody → mconnor
Comment 13•18 years ago
|
||
*** Bug 351150 has been marked as a duplicate of this bug. ***
Comment 14•18 years ago
|
||
The current tabstrip color (grey) matches the WinXP style Luna Silver fine (because the toolbars are also grey ), but does not go together with WinXP style Luna Blue (where the toolbars are beige by default).
Reporter | ||
Comment 15•18 years ago
|
||
(In reply to comment #14)
FWIW, this has become a meta bug, please see blocking bugs.
Assignee | ||
Comment 16•18 years ago
|
||
Fixed now. We might still tweak, but this is resolved as long as it doesn't regress.
Keywords: fixed1.8.1
Assignee | ||
Updated•18 years ago
|
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•