Closed
Bug 449832
Opened 16 years ago
Closed 16 years ago
Color mismatch between unified toolbar and selected tab after bug 439354
Categories
(Firefox :: Theme, defect)
Tracking
()
RESOLVED
FIXED
Firefox 3.1b1
People
(Reporter: mstange, Assigned: mstange)
References
Details
Attachments
(2 files, 3 obsolete files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
In bug 439354 I've changed the unified toolbar to be drawn by a -moz-appearance and I've made the colors darker to match those of native Cocoa applications.
Now the selected tab's color is too light - when the bookmarks bar is hidden, there's a color mismatch.
And there's another problem: When color management is turned on, the tab images are color-corrected, but the unified toolbar is not.
These problems can be solved by using a platform color for the tab's color. Platform colors aren't color-corrected. I'll file a bug for creating the platform colors -moz-mac-chrome-active and -moz-mac-chrome-inactive soon.
This platform color will be set as the background color for the tabs.
Then the tab images need to be changed to be transparent so that the platform color shines through in the right places.
Finally, a trick must be applied to ensure that the tab's shadow is still transparent (and not filled with the platform color) in order to get nice-looking tab drag images (see also bug 408294). I'll solve that with border-radius and negative margins.
Assignee | ||
Comment 1•16 years ago
|
||
The platform colors bug is bug 449833.
Assignee | ||
Comment 2•16 years ago
|
||
This patch also contains the new images and should be applied by adding it to the hg tab queue.
The images tab-left.png, tab-middle.png, tab-right.png and bookmark_toolbar_background now only consist of the colors black and white with different alpha transparency levels.
This also has the advantages of being more friendly to Personas and not needing different images for the active and inactive states of the window (so the flash that occurs when the window is unfocused the first time is avoided).
Keeping the tab shadows free of the background color works like this:
- I set margins on .tabbrowser-tab so that its background color exactly under
the intended opaque part of the tab image and
- then I neutralize these margins with negative margins on the tab images
(so that the shadow is pulled out).
These margins are 2px on the left and 3px on the bottom and the right.
The patch also includes a minor design change: the gradient in the bookmarks bar has become a little softer.
Assignee | ||
Comment 3•16 years ago
|
||
Attachment #332996 -
Attachment is obsolete: true
Assignee | ||
Comment 4•16 years ago
|
||
Assignee | ||
Comment 5•16 years ago
|
||
fix v0.2 apparently didn't contain the new bookmarks bar image
Attachment #333017 -
Attachment is obsolete: true
Attachment #333466 -
Flags: review?(mconnor)
Updated•16 years ago
|
Attachment #333466 -
Flags: review?(mconnor) → review+
Assignee | ||
Comment 6•16 years ago
|
||
This patch applies on top of the other patch.
It changes two things:
- RTL tab margins fix
- The background and border colors of the Bookmarks toolbar are moved into a
general rule in global/toolbar.css. The border colors are tweaked a little.
Attachment #335435 -
Flags: review?(mconnor)
Assignee | ||
Comment 7•16 years ago
|
||
Comment on attachment 333466 [details] [diff] [review]
fix v0.2.1 [checked in]
http://hg.mozilla.org/mozilla-central/rev/b490bfe2180a
Attachment #333466 -
Attachment description: fix v0.2.1 → fix v0.2.1 [checked in]
Assignee | ||
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1b1
Assignee | ||
Comment 8•16 years ago
|
||
Comment on attachment 335435 [details] [diff] [review]
follow-up patch to fix rtl and normal toolbars
I've moved the follow-up fix to bug 455787.
Attachment #335435 -
Attachment is obsolete: true
Attachment #335435 -
Flags: review?(mconnor)
You need to log in
before you can comment on or make changes to this bug.
Description
•