Closed
Bug 985186
Opened 11 years ago
Closed 11 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•11 years ago
|
||
Attachment #8393238 -
Flags: review?(jfkthame)
Comment 2•11 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•11 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•11 years ago
|
Attachment #8393238 -
Flags: review?(jfkthame) → review+
Assignee | ||
Updated•11 years ago
|
status-firefox28:
--- → affected
status-firefox29:
--- → affected
status-firefox30:
--- → affected
status-firefox31:
--- → affected
Keywords: checkin-needed
Assignee | ||
Comment 4•11 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•11 years ago
|
||
Flags: in-testsuite+
Keywords: checkin-needed
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Updated•11 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•11 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•