Closed
Bug 1468864
Opened 6 years ago
Closed 6 years ago
Sideloading indicator styling is broken
Categories
(Toolkit :: Add-ons Manager, defect)
Tracking
()
VERIFIED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox-esr60 | --- | unaffected |
firefox60 | --- | unaffected |
firefox61 | --- | unaffected |
firefox62 | --- | verified |
People
(Reporter: aswan, Assigned: jaws)
References
Details
(Keywords: regression)
Attachments
(2 files)
The sideloading indicator used to be orange but it has turned black.
It looks like perhaps this rule is inadvertently propagating to the badge?
https://searchfox.org/mozilla-central/rev/42930ab9634ebf3f62aed60f7d1c1bf25c0bf00c/browser/themes/shared/toolbarbutton-icons.inc.css#18
Assignee | ||
Comment 2•6 years ago
|
||
Tim, I thought https://hg.mozilla.org/mozilla-central/rev/0beab09f8058#l6.15 was supposed to handle this?
Flags: needinfo?(jaws) → needinfo?(ntim.bugs)
Assignee | ||
Comment 3•6 years ago
|
||
Specifically, the fallbacks in `fill="context-fill #FFBF00" stroke="context-stroke #fff"`.
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(jaws)
Assignee | ||
Comment 4•6 years ago
|
||
Oh, nevermind, yeah the link that Andrew posted is what is setting the fill instead of the fallback color getting used. I'll write up a patch.
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Flags: needinfo?(ntim.bugs)
Flags: needinfo?(jaws)
Comment hidden (mozreview-request) |
Comment 6•6 years ago
|
||
mozreview-review |
Comment on attachment 8985544 [details]
Bug 1468864 - Set the fill and stroke for warning.svg in a few places that inherit fill and stroke colors on from toolbarbuttons.
https://reviewboard.mozilla.org/r/251156/#review257444
::: browser/themes/shared/customizableui/panelUI.inc.css:95
(Diff revision 1)
> + -moz-context-properties: fill, stroke;
> + fill: #FFBF00;
> + stroke: #fff;
-moz-context-properties: none; should just work instead of redefining the fallbacks already in the SVG
::: browser/themes/shared/customizableui/panelUI.inc.css:453
(Diff revision 1)
> + -moz-context-properties: fill, stroke;
> + fill: #FFBF00;
> + stroke: #fff;
> }
>
> :root[lwt-popup-brighttext] .addon-banner-item::after {
> - -moz-context-properties: fill, stroke;
Is this change needed ?
Attachment #8985544 -
Flags: review?(ntim.bugs)
Comment hidden (mozreview-request) |
Comment 8•6 years ago
|
||
mozreview-review |
Comment on attachment 8985544 [details]
Bug 1468864 - Set the fill and stroke for warning.svg in a few places that inherit fill and stroke colors on from toolbarbuttons.
https://reviewboard.mozilla.org/r/251156/#review257598
::: browser/themes/shared/customizableui/panelUI.inc.css:94
(Diff revision 2)
> background: transparent url(chrome://browser/skin/warning.svg) no-repeat center;
> + background-size: contain;
Please move this to the shorthand.
::: browser/themes/shared/customizableui/panelUI.inc.css:98
(Diff revision 2)
> height: 13px;
> background: transparent url(chrome://browser/skin/warning.svg) no-repeat center;
> + background-size: contain;
> + box-shadow: none;
> + border-radius: 0;
> + -moz-context-properties: none;
Please add a comment explaining this.
Attachment #8985544 -
Flags: review?(ntim.bugs)
Comment hidden (mozreview-request) |
Comment 10•6 years ago
|
||
mozreview-review |
Comment on attachment 8985544 [details]
Bug 1468864 - Set the fill and stroke for warning.svg in a few places that inherit fill and stroke colors on from toolbarbuttons.
https://reviewboard.mozilla.org/r/251156/#review258002
Attachment #8985544 -
Flags: review?(ntim.bugs) → review+
Comment 11•6 years ago
|
||
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/f33185af8502
Set the fill and stroke for warning.svg in a few places that inherit fill and stroke colors on from toolbarbuttons. r=ntim
Comment 12•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
status-firefox60:
--- → unaffected
status-firefox-esr52:
--- → unaffected
status-firefox-esr60:
--- → unaffected
Comment 13•6 years ago
|
||
Issue reproduced in Firefox 62.0a1 (20180614220111).
Retested and verified in Firefox 62.0a1 (20180621220127) on Windows 10 64Bit, MacOS 10.13.4.
Updated•6 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•