Closed
Bug 1156996
Opened 9 years ago
Closed 3 years ago
Bullet is not aligned with the text on vertical writing mode
Categories
(Core :: Layout: Text and Fonts, defect)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
91 Branch
People
(Reporter: xidorn, Assigned: MatsPalmgren_bugz)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Compare these two page: data:text/html,<html style="writing-mode: vertical-lr"><ol style="list-style: decimal inside; padding: 0"><li>123 data:text/html,<html style="writing-mode: vertical-lr">1. 123 When the writing-mode is horizontal, the two pages are identical. However, on vertical text, there is a small offset on the bullet text. This offset may vary among fonts. I met the same problem when I works on bug 1117227. I could not find a sensible way to write reftest for that because of this offset.
Reporter | ||
Comment 1•9 years ago
|
||
jfkthame, could you have a look at this issue?
Flags: needinfo?(jfkthame)
Comment 2•9 years ago
|
||
I think you can avoid this problem, at least for the purpose of your reftests, by specifying text-orientation:sideways-right. The reason is that with text-orientation:mixed (the initial value), vertical text uses a dominant center baseline (better for ideographic text, but can result in issues with alignment of alphabetic runs.)
Flags: needinfo?(jfkthame)
Reporter | ||
Updated•9 years ago
|
Assignee | ||
Comment 3•3 years ago
|
||
Bug 1542807 fixed this bug. I'll add some WPTs for this...
Assignee: nobody → mats
Depends on: 1542807
Assignee | ||
Comment 4•3 years ago
|
||
Updated•3 years ago
|
Attachment #9227310 -
Attachment description: Bug 1156996 - Add WPT checking baseline alignement of 'list-style: decimal inside' ::marker pseudos. r=#layout-reviewers → Bug 1156996 - Add WPT checking baseline alignment of 'list-style: decimal inside' ::marker pseudos. r=#layout-reviewers
Pushed by mpalmgren@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/38eb426a8439 Add WPT checking baseline alignment of 'list-style: decimal inside' ::marker pseudos. r=TYLin
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/29472 for changes under testing/web-platform/tests
Comment 7•3 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox91:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
Upstream PR merged by moz-wptsync-bot
Updated•3 years ago
|
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•