Bug 80072 - [chromium] Japanese vertical text displayed in uneven spacing
Summary: [chromium] Japanese vertical text displayed in uneven spacing
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-01 16:54 PST by Xiaomei Ji
Modified: 2013-04-15 06:24 PDT (History)
3 users (show)

See Also:


Attachments
spacing and parenthesis in cr-linux (176.67 KB, image/png)
2012-03-01 16:54 PST, Xiaomei Ji
no flags Details
spacing and parenthesis in cr-mac (153.17 KB, image/png)
2012-03-01 16:54 PST, Xiaomei Ji
no flags Details
screenshot (147.72 KB, image/png)
2012-03-07 00:59 PST, Koan-Sin Tan
no flags Details
safari screenshot (123.05 KB, image/png)
2012-03-08 14:13 PST, Xiaomei Ji
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Xiaomei Ji 2012-03-01 16:54:04 PST
Created attachment 129773 [details]
spacing and parenthesis in cr-linux

load http://dl.dropbox.com/u/64492542/epub3/verttest.html.

right under the title, you could see that the beginning parenthesis "(" is displayed as is without rotation for vertical text, while ending parenthesis is.

Also, if you look at the text closely, you could see that text are bit crowded. Looks like after ruby signs appear, the lines without ruby sign are closely adjacent to the previous line, instead of having a space (with the same width as where ruby signs are displayed) in between.

I attached the snapshot for both mac and linux for comparison.
Comment 1 Xiaomei Ji 2012-03-01 16:54:31 PST
Created attachment 129774 [details]
spacing and parenthesis in cr-mac
Comment 2 Koan-Sin Tan 2012-03-07 00:16:14 PST
The parenthesis problem was caused by a typo. The opening one is a FULLWIDTH LEFT PARENTHESIS (U+FF08) instead of U+0028. To display it correctly, you need a font with proper vert/vrt2 tables.
Comment 3 Koan-Sin Tan 2012-03-07 00:43:55 PST
regarding the space problem, on my system, it doesn't look like the space between vertical lines is gone.
Comment 4 Koan-Sin Tan 2012-03-07 00:59:17 PST
Created attachment 130564 [details]
screenshot 

it seems to me line space is there
Comment 5 Xiaomei Ji 2012-03-08 14:13:16 PST
Created attachment 130903 [details]
safari screenshot

I attached screenshot from Safari as well.
If you look closely on the cr-mac and Safari screenshots,  you will see the line space between each line is fixed width.

If you look at your screenshot, you can see the line space between each line is not fixed.
If you look at the 1st and 2nd line in main content, there is wider space in between. It is approximately the same wide as the line space between 2nd and 3rd line (while 3rd line has ruby).
If you look at space between line 4th and 5th, you can see the space in between is narrower.

What I heard from Japanese and traditional Chinese speaker is that the space between lines should be fixed.
Comment 6 Koji Ishii 2012-04-30 19:12:46 PDT
I suspect "wrong parenthesis" part is dup of bug 69282 for cr-linux. The plan is to finish Apple Win port in bug 48459, port it to cr-win in bug 51450, and bashi@google.com will port it to bug 69282. It's a bit long way though.

Line pitch with ruby seems to be a separate bug to me. Probably you should change this bug to the line pitch with ruby. For regular Japanese publications, author usually set line-height to 1.8 or 2, and the bug will not appear. It only happens when line-height isn't big enough to show ruby.
Comment 7 Xiaomei Ji 2012-05-01 10:03:23 PDT
(In reply to comment #6)
> I suspect "wrong parenthesis" part is dup of bug 69282 for cr-linux. The plan is to finish Apple Win port in bug 48459, port it to cr-win in bug 51450, and bashi@google.com will port it to bug 69282. It's a bit long way though.

I do not quite understand why this is related to  bug 48459.
vertical display is not supported in Windows port yet.
But it is supported in cr-linux. As to the wrong parenthesis, if there is no vert or vrt2 table, we will need to create a separate mapping table, is it?
Comment 8 Koji Ishii 2012-05-02 02:09:49 PDT
I see your point, I thought you're using fonts with vert, and I'm not very aware of the current state of cr-linux port.

So there are two issues:
1. Support vertical flow with fonts without vert table (note that the current CSS spec doesn't use vrt2; just vert)
2. Line height is calculated incorrectly when there's ruby and line height isn't big enough to hold ruby

I think you should file two separate bugs. The former is a kind of feature addition. How common is such font? Right now, the CSS spec doesn't support such fonts and therefore it will likely be webkit extension to the spec (remember, the spec is still WD.) You may want to discuss at www-style@w3.org to allow UA to support such fonts if you think you really need it, so that it becomes part of the official spec.

The later is a separate bug in ruby code, and it reproduces on other ports as well. It's hard to work on if the two are mixed in single bug.
Comment 9 Xiaomei Ji 2012-05-02 10:35:03 PDT
I've changed the bug title. And the 1st issue is probably a dup of bug 69282. I will file a separate one if I found out that it is not a dup.