Closed
Bug 1388904
Opened 7 years ago
Closed 7 years ago
stylo: <ruby> at line ends may not have correct justification applied
Categories
(Core :: CSS Parsing and Computation, defect, P2)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: manishearth, Assigned: xidorn)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/html
|
Details |
In gecko, the attached test case will read "東 京" in kanji with ruby on top. The "東" and "京" will be at the 1/3 and 2/3 mark of the page.
In stylo, the "東" and "京" will be at opposite ends of the page, but the ruby will be in the same spot.
I thought this might be due to us not handling mTextAlignLastUnsafe, but that's not getting set on this page.
This causes ./layout/reftests/writing-mode/1135361-ruby-justify-1.html
Reporter | ||
Updated•7 years ago
|
Attachment #8895576 -
Attachment mime type: text/plain → text/html
Reporter | ||
Comment 1•7 years ago
|
||
I also verified that the getComputedStyle() trees are identical (which only points more strongly to something weird going on with the disabled unsafe/true value support), but that's not the case either.
Assignee | ||
Comment 2•7 years ago
|
||
So this is because the SHOULD_SUPPRESS_LINEBREAK flag is not correctly set to the text frame in this case, so it breaks the shortcut check at https://searchfox.org/mozilla-central/rev/e5b13e6224dbe3182050cf442608c4cb6a8c5c55/layout/generic/nsLineLayout.cpp#3133,3142
So in this case, inheriting flags doesn't work, because the text frame inherits from <ruby>, which doesn't have the flag we want. We need to do an additional check in StyleAdjuster::adjust_for_text() to apply this flag when its parent has any ruby display type.
Assignee: nobody → xidorn+moz
Priority: P3 → P2
Summary: stylo: text-align-last doesn't work with <ruby> inside <p> → stylo: <ruby> at line ends may not have correct justification applied
Assignee | ||
Comment 3•7 years ago
|
||
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aeef23d80cf2
Update reftest expectation.
Comment 5•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•