Bug 279217

Summary: text-emphasis-position default for vertical typographic modes is wrong
Product: WebKit Reporter: fantasai <fantasai.bugs>
Component: Layout and RenderingAssignee: fantasai <fantasai.bugs>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
URL: https://software.hixie.ch/utilities/js/live-dom-viewer/?%3C!DOCTYPE%20html%3E%0A%3Chtml%20style%3D%22writing-mode%3A%20vertical-rl%22%3E%0A%3Cstyle%3Eem%20%7B%20text-emphasis%3A%20dot%3B%20%7D%3C%2Fstyle%3E%0A%3Cp%20style%3D%22text-emphasis-position%3A%20under%22%3E%3Cem%3Eunder%3C%2Fem%3E%20%3D%20right%3C%2Fp%3E%0A%3Cp%20style%3D%22text-emphasis-position%3A%20over%22%3E%3Cem%3Eunder%3C%2Fem%3E%20%3D%20right%3C%2Fp%3E%0A%3Cp%20style%3D%22text-emphasis-position%3A%20left%20under%22%3E%3Cem%3Eleft%20under%3C%2Fem%3E%20%3D%20left%3C%2Fp%3E%0A%3Cp%20style%3D%22text-emphasis-position%3A%20right%20under%22%3E%3Cem%3Eright%20under%3C%2Fem%3E%20%3D%20right%3C%2Fp%3E%0A%3Cp%20style%3D%22text-emphasis-position%3A%20left%20over%22%3E%3Cem%3Eleft%20over%3C%2Fem%3E%20%3D%20left%3C%2Fp%3E%0A%3Cp%20style%3D%22text-emphasis-position%3A%20right%20over%22%3E%3Cem%3Eright%20over%3C%2Fem%3E%20%3D%20right%3C%2Fp%3E

fantasai
Reported 2024-09-05 12:48:55 PDT
https://www.w3.org/TR/css-text-decor-4/#text-emphasis-position-property > If [ right | left ] is omitted, it defaults to right. We currently interpret over/under as for sideways writing modes instead of defaulting to right. See testcase.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2024-09-05 13:14:32 PDT
<rdar://problem/135366186>
fantasai
Comment 2 2024-09-05 14:40:45 PDT
Pull request: https://github.com/WebKit/WebKit/pull/33210
EWS
Comment 3 2024-09-06 11:15:13 PDT
Committed 283275@main (29304d7c9119): <https://commits.webkit.org/283275@main> Reviewed commits have been landed. Closing PR #33210 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.