Closed
Bug 1388907
Opened 7 years ago
Closed 7 years ago
Support fill="context-fill" for test pilot extensions
Categories
(Core :: SVG, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jkt, Assigned: jkt)
References
Details
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
jwatt
:
review+
gchang
:
approval-mozilla-beta+
|
Details |
Currently Testpilot and containers would like to ship with context-fill icons to match the browser chrome. Whilst we are at it we should support @mozilla.com as voice fill uses this. This might not be the perfect long term solution as we would like to use the signing key, however this is something we would like to uplift with minimal risk.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Updated•7 years ago
|
Attachment #8895607 -
Flags: review?(dao+bmo) → review?(jwatt)
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8895607 [details] Bug 1388907 - Add @mozilla.com and @testpilot- extension identifiers for SVG context-fill icons. https://reviewboard.mozilla.org/r/166832/#review172098
Attachment #8895607 -
Flags: review?(jwatt) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/cf770754be29 Add @mozilla.com and @testpilot- extension identifiers for SVG context-fill icons. r=jwatt
Keywords: checkin-needed
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/cf770754be29
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8895607 [details] Bug 1388907 - Add @mozilla.com and @testpilot- extension identifiers for SVG context-fill icons. Approval Request Comment [Feature/Bug causing the regression]: N/A [User impact if declined]: Test pilot extensions won't be able to match native icons. [Is this code covered by automated tests?]: N/A [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: Steps to reproduce are install a an addon with context-fill, you could use the branch here: https://github.com/jonathanKingston/testpilot-containers/commit/78b5de3b44c53870b387bd978673fd0074de6907 [List of other uplifts needed for the feature/fix]: N/A [Is the change risky?]: No [Why is the change risky/not risky?]: It widens the exposure to context fill icons only. [String changes made/needed]: N/A
Attachment #8895607 -
Flags: approval-mozilla-beta?
Updated•7 years ago
|
status-firefox56:
--- → affected
Comment 7•7 years ago
|
||
Calling out that uplifting this is important for Test Pilot since we've moved our browserAction to a webExtension
Comment 8•7 years ago
|
||
Comment on attachment 8895607 [details] Bug 1388907 - Add @mozilla.com and @testpilot- extension identifiers for SVG context-fill icons. For test pilot. Beta56+. Should be in 56.0b3.
Attachment #8895607 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 9•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/efabe9b3a88e
Updated•7 years ago
|
Component: Theme → SVG
Product: Firefox → Core
Target Milestone: Firefox 57 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•