Closed
Bug 1185725
Opened 9 years ago
Closed 9 years ago
The update-status badge on #PanelUI-menu-button isn't visible anymore
Categories
(Firefox :: Toolbars and Customization, defect)
Firefox
Toolbars and Customization
Tracking
()
RESOLVED
FIXED
Firefox 42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: bgrins, Assigned: neil)
References
Details
(Keywords: regression)
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
After Bug 1029937, the markup for toolbarbutton-badges changed. But there is still one instance of toolbarbutton-badge::after referenced in browser/themes/shared/customizableui/panelUIOverlay.inc.css: http://mxr.mozilla.org/mozilla-central/search?string=toolbarbutton-badge%3A%3Aafter.
This means that the green arrow shown when there is a pending update isn't visible.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1029937#c44.
Reporter | ||
Comment 1•9 years ago
|
||
Moving the discussion from Bug 1029937 to here. As per my review comment, there is an issue on osx with an unfocused window.
Here's a screenshot with the patch from Bug 1029937 applied in OSX showing an issue with the icon being clipped when the window is unfocused. Looks like there is a 1px border applied to .toolbarbutton-badge:-moz-window-inactive from toolbarbuttons.css which causes the issue. It might be simplest to just apply border: none to the #PanelUI-menu-button[update-status="succeeded"] > .toolbarbutton-badge-stack > .toolbarbutton-badge rule.
Flags: needinfo?(neil)
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to Brian Grinstead from comment #1)
> Looks like there is a 1px border applied to .toolbarbutton-badge:-moz-window-inactive
(Yeah, I did wonder about that at the time when writing the patch for bug 1029937, it seems odd to change the size of UI depending on whether the window is active or not.)
Flags: needinfo?(neil)
Assignee | ||
Comment 3•9 years ago
|
||
Reporter | ||
Updated•9 years ago
|
Attachment #8636573 -
Flags: review?(bgrinstead) → review+
Assignee | ||
Comment 4•9 years ago
|
||
Comment 5•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
You need to log in
before you can comment on or make changes to this bug.
Description
•