Closed
Bug 1474836
Opened 6 years ago
Closed 6 years ago
Font editor: do not invoke on pseudo-elements
Categories
(DevTools :: Inspector, defect, P2)
DevTools
Inspector
Tracking
(firefox63 fixed)
RESOLVED
FIXED
Firefox 63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: rcaliman, Assigned: rcaliman)
References
Details
Attachments
(2 files)
As long as property value changes are written exclusively to the selected element's inline style attribute, pseudo-elements cannot be supported by the font editor. When a pseudo-element like ::before or ::after is selected, show the font editor empty state with a warning message with the reason.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8991240 [details] Bug 1474836 - (Part 1) Externalize font editor empty state warning message to Redux store. https://reviewboard.mozilla.org/r/256196/#review263060 Code analysis found 1 defect in this patch: - 1 defect found by mozlint You can run this analysis locally with: - `./mach lint path/to/file` (JS/Python) If you see a problem in this automated review, please report it here: http://bit.ly/2y9N9Vx ::: devtools/client/inspector/fonts/fonts.js:34 (Diff revision 1) > resetFontEditor, > updateAxis, > updateCustomInstance, > updateFontEditor, > updateFontProperty, > + updateWarningMessage, Error: 'updateWarningMessage' is assigned a value but never used. [eslint: no-unused-vars]
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 6•6 years ago
|
||
mozreview-review |
Comment on attachment 8991241 [details] Bug 1474836 - (Part 2) Show editor empty state with custom warning for pseudo-elements. https://reviewboard.mozilla.org/r/256198/#review263116
Attachment #8991241 -
Flags: review?(gl) → review+
Comment 7•6 years ago
|
||
mozreview-review |
Comment on attachment 8991240 [details] Bug 1474836 - (Part 1) Externalize font editor empty state warning message to Redux store. https://reviewboard.mozilla.org/r/256196/#review263118
Attachment #8991240 -
Flags: review?(gl) → review+
Pushed by rcaliman@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0a357b54899d (Part 1) Externalize font editor empty state warning message to Redux store. r=gl https://hg.mozilla.org/integration/autoland/rev/e4d608a4a833 (Part 2) Show editor empty state with custom warning for pseudo-elements. r=gl
Comment 10•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0a357b54899d https://hg.mozilla.org/mozilla-central/rev/e4d608a4a833
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Updated•6 years ago
|
QA Contact: catalin.sasca
Updated•3 years ago
|
Component: Inspector: Fonts → Inspector
You need to log in
before you can comment on or make changes to this bug.
Description
•