Closed
Bug 616680
Opened 15 years ago
Closed 15 years ago
Clean up app button styling in browser-aero.css
Categories
(Firefox :: Theme, defect)
Tracking
()
RESOLVED
FIXED
Firefox 4.0b8
People
(Reporter: dao, Assigned: dao)
Details
Attachments
(1 file, 1 obsolete file)
|
3.61 KB,
patch
|
Gavin
:
review+
dietrich
:
approval2.0+
|
Details | Diff | Splinter Review |
From bug 606160 comment 33:
> Note that windows-compositor doesn't imply windows-default-theme and the
> negative margin actually needs both, since we set the extra border only for
> windows-default-theme. A better solution might be to change the border to be
> added for everything but windows-classic in browser-aero.css:
>
> @media not all and (-moz-windows-classic) {
> #appmenu-button {
> border: ...
> margin-bottom: 1px; /* compensate white outer border */
> }
> }
>
> @media all and (-moz-windows-compositor) {
> #appmenu-button {
> margin-bottom: -1px; /* compensate white outer border */
> }
> }
The attached patch does that.
Attachment #495239 -
Flags: review?(gavin.sharp)
|
Assignee | |
Comment 1•15 years ago
|
||
updated to tip
Attachment #495239 -
Attachment is obsolete: true
Attachment #495473 -
Flags: review?(gavin.sharp)
Attachment #495239 -
Flags: review?(gavin.sharp)
|
|
Assignee | |
Updated•15 years ago
|
Attachment #495473 -
Flags: review?(jmathies)
|
||
Comment 2•15 years ago
|
||
Comment on attachment 495473 [details] [diff] [review]
patch
rs=me...
Attachment #495473 -
Flags: review?(gavin.sharp) → review+
|
|
Assignee | |
Updated•15 years ago
|
Attachment #495473 -
Flags: review?(jmathies) → approval2.0?
|
||
Updated•15 years ago
|
Attachment #495473 -
Flags: approval2.0? → approval2.0+
|
|
Assignee | |
Comment 3•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b8
You need to log in
before you can comment on or make changes to this bug.
Description
•