Closed
Bug 1224013
Opened 9 years ago
Closed 9 years ago
Render text emphasis marks outside ruby
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
References
(Blocks 2 open bugs, )
Details
Attachments
(4 files)
I'd like to separate this issue to an independent bug, because there is an unsolved question on this behavior, see: https://lists.w3.org/Archives/Public/www-style/2015Nov/0193.html
Depending on the answer there, the implementation could be easy or super complicated that we may only want an imperfect impl.
Assignee | ||
Updated•9 years ago
|
Blocks: css-text-emphasis
Assignee | ||
Comment 1•9 years ago
|
||
Bug 1224013 part 1 - Enable text-emphasis in UA sheets.
The only usage of text-emphasis in UA sheets is to clear it on ruby text. Making
it enabled in UA sheets allows that rule to take effect without additional code
to reload html.css when the pref changes.
Attachment #8694160 -
Flags: review?(dbaron)
Assignee | ||
Comment 2•9 years ago
|
||
Bug 1224013 part 2 - Move leading fix to ruby base container.
Per spec, the additional leading is added on side of ruby base container, not
ruby container. This could lead to some trivial behavior changes on edge cases.
Attachment #8694161 -
Flags: review?(dholbert)
Assignee | ||
Comment 3•9 years ago
|
||
Bug 1224013 part 3 - Render text-emphasis outside ruby.
Attachment #8694162 -
Flags: review?(jfkthame)
Assignee | ||
Comment 4•9 years ago
|
||
Bug 1224013 part 4 - Add reftests for text-emphasis with ruby.
Attachment #8694163 -
Flags: review?(jfkthame)
Comment 5•9 years ago
|
||
Comment on attachment 8694160 [details]
MozReview Request: Bug 1224013 part 1 - Enable text-emphasis in UA sheets.
https://reviewboard.mozilla.org/r/26667/#review24207
It might be worth putting a note on the bug to enable these properties that you can remove these flags when you remove the pref. (If there's not a bug to enable them already, there should be... with bugs blocking it.)
Attachment #8694160 -
Flags: review?(dbaron) → review+
Comment 6•9 years ago
|
||
Comment on attachment 8694161 [details]
MozReview Request: Bug 1224013 part 2 - Move leading fix to ruby base container.
https://reviewboard.mozilla.org/r/26669/#review24409
Attachment #8694161 -
Flags: review?(dholbert) → review+
Comment 7•9 years ago
|
||
Comment on attachment 8694162 [details]
MozReview Request: Bug 1224013 part 3 - Render text-emphasis outside ruby.
https://reviewboard.mozilla.org/r/26671/#review24429
Attachment #8694162 -
Flags: review?(jfkthame) → review+
Comment 8•9 years ago
|
||
Comment on attachment 8694163 [details]
MozReview Request: Bug 1224013 part 4 - Add reftests for text-emphasis with ruby.
https://reviewboard.mozilla.org/r/26673/#review24431
Attachment #8694163 -
Flags: review?(jfkthame) → review+
Assignee | ||
Comment 9•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9dfd459c839628fc852a8beaf2fb599b28269dd2
Bug 1224013 part 1 - Enable text-emphasis in UA sheets. r=dbaron
https://hg.mozilla.org/integration/mozilla-inbound/rev/15113b4231c70ccbc1ce420fd03f1984c5b89d22
Bug 1224013 part 2 - Move leading fix to ruby base container. r=dholbert
https://hg.mozilla.org/integration/mozilla-inbound/rev/5c44af8f7d6ee6319fb5d82970c8811383036ea9
Bug 1224013 part 3 - Render text-emphasis outside ruby. r=jfkthame
https://hg.mozilla.org/integration/mozilla-inbound/rev/b0b15e5ed2712aa171dd39ba98a42e831f0d7b1d
Bug 1224013 part 4 - Add reftests for text-emphasis with ruby. r=jfkthame
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → quanxunzhen
Assignee | ||
Comment 10•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/835de4b7bcb2b9016ef425db25878929c28a9448
Backed out 8 changesets (bug 1225018, bug 1229278, bug 1224013) for reftest failures on CLOSED TREE
Assignee | ||
Comment 11•9 years ago
|
||
So part 2 breaks the reftests because when the ruby base container is placed, its required leadings have not been figured out.
I'm going to land the patches except part 2 (also change part 3 to find nearest ruby frame instead of ruby base container).
Assignee | ||
Comment 12•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ddf13e6aa97ef9585b9d06deab3ee9b1e0ce81d3
Bug 1224013 part 1 - Enable text-emphasis in UA sheets. r=dbaron
https://hg.mozilla.org/integration/mozilla-inbound/rev/5f5ca882d872a71622cc305bc25a31fb5fc7ca79
Bug 1224013 part 2 - Render text-emphasis outside ruby. r=jfkthame
https://hg.mozilla.org/integration/mozilla-inbound/rev/3714873643124a5c65c938366db4eba3fc9cbcb4
Bug 1224013 part 3 - Add reftests for text-emphasis with ruby. r=jfkthame
Comment 13•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ddf13e6aa97e
https://hg.mozilla.org/mozilla-central/rev/5f5ca882d872
https://hg.mozilla.org/mozilla-central/rev/371487364312
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•