Closed Bug 1484422 Opened 7 years ago Closed 7 years ago

Clean up fullscreen window controls styling

Categories

(Firefox :: Theme, enhancement, P3)

63 Branch
enhancement

Tracking

()

RESOLVED FIXED
Firefox 63
Tracking Status
firefox63 --- fixed

People

(Reporter: dao, Assigned: dao)

References

Details

Attachments

(1 file)

from bug 1483887 comment 4: > > ::: browser/themes/linux/browser.css > > @@ +153,5 @@ > > > +#minimize-button, > > > +#restore-button, > > > +#close-button { > > > + -moz-appearance: none; > > > + border: none; > > > > I'm not seeing any default border in toolbarbutton.css, is this needed ? > > > > @@ +154,5 @@ > > > +#restore-button, > > > +#close-button { > > > + -moz-appearance: none; > > > + border: none; > > > + margin: 0 !important; > > > > Is there a reason to use !important here ? > > I just copied this from the Windows stylesheet. Chances are we can simplify > this in both places, so I'll file a followup on that.
Assignee: nobody → dao+bmo
Summary: Simplify fullscreen window controls styling → Clean up fullscreen window controls styling
Attached patch patchSplinter Review
Attachment #9002706 - Flags: review?(ntim.bugs)
Attachment #9002706 - Flags: review?(ntim.bugs) → review+
Keywords: checkin-needed
Pushed by ebalazs@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/95b2521e281c Clean up fullscreen window controls styling. r=ntim
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Regressions: 1621632
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: