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)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
91 Branch
Tracking Status
firefox40 --- wontfix
firefox91 --- fixed

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.
jfkthame, could you have a look at this issue?
Flags: needinfo?(jfkthame)
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)
Blocks: writing-mode
No longer blocks: enable-writing-mode-release

Bug 1542807 fixed this bug. I'll add some WPTs for this...

Assignee: nobody → mats
Depends on: 1542807
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
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
Upstream PR merged by moz-wptsync-bot
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: