Clean up fullscreen window controls styling

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P3
normal
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: dao, Assigned: dao)

Tracking

63 Branch
Firefox 63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

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
Posted 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
https://hg.mozilla.org/mozilla-central/rev/95b2521e281c
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
You need to log in before you can comment on or make changes to this bug.