Closed
Bug 1457373
Opened 6 years ago
Closed 6 years ago
Remove pref "layout.css.text-combine-upright-digits.enabled"
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Tracking
()
RESOLVED
FIXED
mozilla62
People
(Reporter: mozbugz, Assigned: mozbugz)
References
Details
Attachments
(1 file)
Spawned from bug 1417761 comment 3 by Xidorn: > ::: modules/libpref/init/all.js:2902 > > // Is support for CSS text-combine-upright: digits 2-4 enabled? > > pref("layout.css.text-combine-upright-digits.enabled", false); > > This pref is no longer effective because stylo doesn't include this. > Could you add a followup patch to remove this pref and any related code as well? And bug 1417761 comment 6: Gerald: > No chance this may instead be re-implemented in Stylo one day? Xidorn: > Implementing the style part is easy, but the layout part is hard, so we are > not going to revive its style part until we decide to work on the layout > part. > > Actually the usefulness of "digits" value as it's currently defined is > unclear either, given it's easy to cause undesired result if not used > carefully. Currently only Edge implemented that, and I don't think Chrome > has plan for this at the moment either.
Updated•6 years ago
|
Priority: -- → P3
Comment hidden (mozreview-request) |
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → gsquelart
Assignee | ||
Updated•6 years ago
|
Attachment #8981064 -
Flags: review?(xidorn+moz)
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8981064 [details] Bug 1457373 - Remove text-combine-upright digits option - https://reviewboard.mozilla.org/r/247184/#review253394
Attachment #8981064 -
Flags: review?(xidorn+moz) → review+
Pushed by gsquelart@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8ecc66673444 Remove text-combine-upright digits option - r=xidorn
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8ecc66673444
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•