Closed
Bug 1919658
Opened 1 year ago
Closed 1 year ago
[css-text-decor] Add 'text-emphasis-position: auto'
Categories
(Core :: Layout: Text and Fonts, defect)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
132 Branch
| Tracking | Status | |
|---|---|---|
| firefox132 | --- | fixed |
People
(Reporter: mozilla-apprentice, Assigned: jfkthame)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(3 files)
A resolution was made for csswg-drafts/#1198.
[css-text-decor] text-underline-position auto in vertical text
- RESOLVED: add auto value for text-emphasis-position, and change the meaning of text-underline-position: auto to care about left vs right in vertical text
Comment 1•1 year ago
|
||
If I understood the discussion correctly, the work for us here is to add a new text-emphasis-position: auto value (now the initial value for that property), which reacts to the language in the same way that text-underline-position: auto already does.
Summary: [css-text-decor] text-underline-position auto in vertical text → [css-text-decor] Add 'text-emphasis-position: auto'
Updated•1 year ago
|
Blocks: css-text-decor-3
| Assignee | ||
Comment 2•1 year ago
|
||
This appears to work as expected in local testing. It'll need some updates to
existing tests, and additional testcases for the new value, so I'll do a second
patch with those.
Updated•1 year ago
|
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
| Assignee | ||
Comment 3•1 year ago
|
||
| Assignee | ||
Comment 4•1 year ago
|
||
Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7e4c6fffa58a
Add support for text-emphasis-position:auto. r=layout-reviewers,emilio
https://hg.mozilla.org/integration/autoland/rev/90acfa4e2255
Update/add parsing tests for text-emphasis-position:auto value. r=layout-reviewers,emilio
https://hg.mozilla.org/integration/autoland/rev/94601aa58c87
Add a WPT reftest for text-emphasis-position:auto behavior. r=layout-reviewers,emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/48294 for changes under testing/web-platform/tests
Updated•1 year ago
|
Keywords: dev-doc-needed
Comment 7•1 year ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/7e4c6fffa58a
https://hg.mozilla.org/mozilla-central/rev/90acfa4e2255
https://hg.mozilla.org/mozilla-central/rev/94601aa58c87
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox132:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
Upstream PR merged by moz-wptsync-bot
Comment 9•1 year ago
•
|
||
Related Pull Requests
Updated•1 year ago
|
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•