Closed
Bug 1488419
Opened 6 years ago
Closed 6 years ago
Font editor: axis value replaces whole font-variation-settings declaration
Categories
(DevTools :: Inspector, defect, P2)
DevTools
Inspector
Tracking
(firefox63 fixed, firefox64 fixed)
RESOLVED
FIXED
Firefox 64
People
(Reporter: rcaliman, Assigned: rcaliman)
References
Details
Attachments
(1 file)
46 bytes,
text/x-phabricator-request
|
gl
:
review+
pascalc
:
approval-mozilla-beta+
|
Details | Review |
Steps: - Go to v-fonts.com - Find the "Renner" font example and inspect the preview text in DevTools - Switch to 3-pane mode with the Fonts panel alongside the CSS rules panel - Drag the Italic slider in the Fonts panel Expected: - Both the "wdth" and the "ital" values show up in the `font-variation-settings` property on the selected element's inline styles. Actual - Sometimes, as you drag the slider, the "ital" axis value replaces the whole value. The behavior is inconsistent. Suspected root cause: Race condition between previewing the changes on the page and writing them to the CSS rule view.
Assignee | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9006255 [details] Bug 1488419 - Re-attach listener for Rule view change after writing new value. r=gl Gabriel [:gl] (ΦωΦ) has approved the revision.
Attachment #9006255 -
Flags: review+
Pushed by rcaliman@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/46f7c652efca Re-attach listener for Rule view change after writing new value. r=gl
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/46f7c652efca
Updated•6 years ago
|
Target Milestone: --- → Firefox 64
Assignee | ||
Comment 5•6 years ago
|
||
Comment on attachment 9006255 [details] Bug 1488419 - Re-attach listener for Rule view change after writing new value. r=gl Approval Request Comment [Feature/Bug causing the regression]: https://bugzilla.mozilla.org/show_bug.cgi?id=1488419 [User impact if declined]: While using the new Font Editor in Devtools some users can lose changes made to the `font-variation-settings` CSS property. [Is this code covered by automated tests?]: No. The current test infrastructure uses OS configurations which don't support variable fonts (current: OSX 10.10, needed: macOS 10.13) [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: No [Why is the change risky/not risky?]: Minimal impact on Font Editor users (subset of DevTools users) editing a variable fonts (even smaller subset of users). The patch fixes a race condition which made the issue very frequent for this subset of users. [String changes made/needed]: None
Attachment #9006255 -
Flags: approval-mozilla-beta?
Comment 6•6 years ago
|
||
Comment on attachment 9006255 [details] Bug 1488419 - Re-attach listener for Rule view change after writing new value. r=gl Uplift approved for 63 beta 5
Attachment #9006255 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/b359fee9b67a
status-firefox63:
--- → fixed
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
•