15630 2007-10-22 15:16:11 -0700 After U+3001, U+3002 (ideographic comma/full stop), lines cannot be broken 2008-02-25 14:28:08 -0800 1 1 1 Unclassified WebKit Layout and Rendering 523.x (Safari 3) PC Windows XP RESOLVED DUPLICATE 17411 http://usstock.jrj.com.cn/xhmt P2 Normal --- 1 jshin webkit-unassigned ap artension mitz oldest_to_newest 59198 0 jshin 2007-10-22 15:16:11 -0700 Due to the problem described in the summary line, the layout at http://usstock.jrj.com.cn/xhmt is broken. WebKit uses ICU line breaking iteratoer, which in turn is based on UAX #14 (Unicode Line Breaking Algorithm). It has the following rule: CL x (AL|NU) where CL includes U+3001 and U+3002 (Ideographic Comma and Full Stop). With the above rule, lines cannot be broken when U+3001 and U+3002 are followed by a Latin letter or a number. As a result, the box at the url given above with the title 59199 1 16810 jshin 2007-10-22 15:19:09 -0700 Created attachment 16810 layout test case two columns should be rendered identically. 59200 2 jshin 2007-10-22 15:24:09 -0700 Hmm my comment #0 got trimmed.... As a result, the box at the url given above with the title 59210 3 jshin 2007-10-22 17:25:23 -0700 Try once more (this time with FF ;-)). The textbox whose title is '美通社简介' is a lot wider than its specified width breaking the layout of the page. A fix is very simple. We have to tailor UAX #14's line breaking property so that U+3001 and U+3002 followed by a Latin letter/number (or more broadly, any character belonging to AL/NU classes) are regarded as a line breaking opportunity. A way to do that is to move those characters from CL class to NS (non-starter) class in ICU's source/data/brkiter/line.txt. For WinSafari, it'd be a simple change, but for Safari on Mac, this may be more involved because it may mean changing the build of ICU shipped with OS X. 59229 4 ddkilzer 2007-10-22 22:56:34 -0700 (In reply to comment #3) > Try once more (this time with FF ;-)). It sounds like you're hitting Bug 14562 (or something similar) when entering text in a text area (which is truncated when sent to the server). Could you please file a new bug on this, stating the version of Safari/WebKit you're using, and steps to reproduce. Thanks! 59441 5 ap 2007-10-25 09:47:05 -0700 I can only reproduce this problem on Windows - Mac (Tiger) works as expected for me. 59545 6 ap 2007-10-26 09:18:33 -0700 Do you know if this has been reported to the Unicode consortium? This rule is new to Unicode 5.0, and doesn't look quite right, as you point out. 59573 7 jshin 2007-10-26 14:44:48 -0700 Yes, I've been in contact with the author of UAX #14 (indirectly). I talked to the author of ICU break iterator and he agreed with me (actually, we sat together and he suggested changing the class of those two to NS). 71030 8 ap 2008-02-18 02:51:21 -0800 Bug 17411 has a patch for this. I'm still unsure whether the Unicode consortium is aware of this issue. ICU is one thing, but the proposed update to UAX #14 at <http://www.unicode.org/reports/tr14/tr14-21.html> seems to be unchanged. 71641 9 artension 2008-02-23 14:33:52 -0800 (In reply to comment #8) I agree. They could be not aware of this issue. I wrote a report about this problem, and sent it to the Unicode ML. http://limechat.net/report/unicode-line-break-problem.html 71673 10 ap 2008-02-23 22:08:00 -0800 Marking as a duplicate, as bug 17411 has an approved fix for this. *** This bug has been marked as a duplicate of 17411 *** 71909 11 jshin 2008-02-25 14:28:08 -0800 (In reply to comment #8) > Bug 17411 has a patch for this. > > I'm still unsure whether the Unicode consortium is aware of this issue. ICU is > one thing, but the proposed update to UAX #14 at > <http://www.unicode.org/reports/tr14/tr14-21.html> seems to be unchanged. In the meantime, they nuked LB30 instead of changing the class for U+3001/3002. A long-term solution is being worked on according to my source. Anyway, on Mac OS X, ICU will always lag behind, I agree that we should fix webkit code (as in bug 17411) . 16810 2007-10-22 15:19:09 -0700 2007-10-22 15:19:09 -0700 layout test case ideo_punc.html text/html 710 jshin PGh0bWw+DQogIDxoZWFkPg0KPG1ldGEgaHR0cC1lcXVpdj0iQ29udGVudC1UeXBlIiBjb250ZW50 PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9dXRmLTgiPg0KPHRpdGxlPnRlc3QgY2FzZSBmb3Igd2Via2l0 IGJ1ZyAxNTYzMDwvdGl0bGU+DQo8L2hlYWQ+DQo8Ym9keT4NCg0KICA8cD5UaGUgY29sdW1uIG9u IHRoZSByaWdodCBoYXMgZXhwbGljaXQgc3BhY2VzIGFmdGVyIGVhY2ggb2YNCiAgVSszMDAxIChp ZGVvZ3JhcGhpYyBjb21tYSkgYW5kIFUrMzAwMiAoaWRlb2dyYXBoaWMgZnVsbCBzdG9wKSB3aGls ZQ0KICB0aGUgbGVmdCBjb2x1bW4gZG9lcyBub3QuIFRoZSBsZWZ0IGFuZCB0aGUgcmlnaHQgY29s dW1ucyBzaG91bGQgYmUgDQogIHJlbmRlcmVkIGlkZW50aWNhbGx5LjwvcD4NCg0KPGRpdiBzdHls ZT0id2lkdGg6MXB4OyBwb3NpdGlvbjphYnNvbHV0ZTsgbGVmdDozMCUiPg0KRXhwZXJ0c+OAgWVX YXRjaChUTSnjgIExMjPjgIFhYmM0NTbjgIExMjNFeHBlcnRz44CCZVdhdGNoKFRNKeOAgjEyM+OA gmFiYzQ1NuOAgjEyMw0KPC9kaXY+DQoNCjxkaXYgc3R5bGU9IndpZHRoOjFweDsgcG9zaXRpb246 YWJzb2x1dGU7IGxlZnQ6NjAlIj4NCkV4cGVydHPjgIEgZVdhdGNoKFRNKeOAgSAxMjPjgIEgYWJj NDU244CBIDEyM0V4cGVydHPjgIIgZVdhdGNoKFRNKeOAgiAxMjPjgIIgYWJjNDU244CCIDEyMw0K PC9kaXY+DQo8L2JvZHk+DQo8L2h0bWw+DQo=