Closed
Bug 985186
Opened 10 years ago
Closed 10 years ago
Unwanted white space for multiple of 10,000 in Korean counter styles
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla31
People
(Reporter: xidorn, Assigned: xidorn)
Details
Attachments
(1 file)
17.80 KB,
patch
|
jfkthame
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
For multiple of 10,000, there is an unwanted space between the number and suffix when using Korean counter styles.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8393238 -
Flags: review?(jfkthame)
Comment 2•10 years ago
|
||
[Reclassifying to Core:Layout. Bugs in the CSS Parsing & Computation component are for changes/issues in code within /layout/style/, generally.]
Component: CSS Parsing and Computation → Layout
Assignee | ||
Comment 3•10 years ago
|
||
(In reply to Daniel Holbert [:dholbert] from comment #2) > [Reclassifying to Core:Layout. Bugs in the CSS Parsing & Computation > component are for changes/issues in code within /layout/style/, generally.] This code will be moved to /layout/style/ in bug 966166.
Updated•10 years ago
|
Attachment #8393238 -
Flags: review?(jfkthame) → review+
Assignee | ||
Updated•10 years ago
|
status-firefox28:
--- → affected
status-firefox29:
--- → affected
status-firefox30:
--- → affected
status-firefox31:
--- → affected
Keywords: checkin-needed
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8393238 [details] [diff] [review] patch [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 934072 User impact if declined: may produce unwanted result if any of the affected counter styles is used with the some ordinal Testing completed (on m-c, etc.): related reftests have been updated Risk to taking this patch (and alternatives if risky): not risky String or IDL/UUID changes made by this patch: n/a
Attachment #8393238 -
Flags: approval-mozilla-beta?
Attachment #8393238 -
Flags: approval-mozilla-aurora?
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/04d8b49bdd18
Flags: in-testsuite+
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/04d8b49bdd18
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Updated•10 years ago
|
Attachment #8393238 -
Flags: approval-mozilla-beta?
Attachment #8393238 -
Flags: approval-mozilla-beta+
Attachment #8393238 -
Flags: approval-mozilla-aurora?
Attachment #8393238 -
Flags: approval-mozilla-aurora+
Comment 7•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/0d083ae6eacc https://hg.mozilla.org/releases/mozilla-beta/rev/26f12c4bb221
You need to log in
before you can comment on or make changes to this bug.
Description
•