Closed
Bug 1110266
Opened 10 years ago
Closed 10 years ago
units that depend on font metrics (such as 'ch') should be derived from horizontal font metrics when writing-mode is vertical but text-orientation is sideways
Categories
(Core :: Layout: Text and Fonts, defect)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: jfkthame, Assigned: jfkthame)
Details
Attachments
(1 file)
1.23 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
Compare:
data:text/html;charset=utf-8,
<div style="font-family:monospace">
<div style="width:9ch; height:9ch; border:1px solid black">123456789</div>
data:text/html;charset=utf-8,
<div style="font-family:monospace; writing-mode:vertical-lr; text-orientation:sideways-right">
<div style="width:9ch; height:9ch; border:1px solid black">123456789</div>
IMO, these should both produce the same size of square <div>, just large enough to fit the line of digits.
Note that it's expected that without text-orientation, as in
data:text/html;charset=utf-8,
<div style="font-family:monospace; writing-mode:vertical-lr">
<div style="width:9ch; height:9ch; border:1px solid black">123456789</div>
we get a larger square, because its size will be based on 9 characters of upright vertical text (consider if the ASCII digits were replaced by fullwidth ones).
Webkit appears to behave in this way, too.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8535084 -
Flags: review?(smontagu)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Comment 2•10 years ago
|
||
Comment on attachment 8535084 [details] [diff] [review]
Use horizontal-font metrics as basis for CSS units when text-orientation is sideways
Review of attachment 8535084 [details] [diff] [review]:
-----------------------------------------------------------------
Makes sense
Attachment #8535084 -
Flags: review?(smontagu) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Target Milestone: --- → mozilla37
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•