Bug 27195

Summary: Text should wrap after curly quotes, not before
Product: WebKit Reporter: iain.dalton
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WORKSFORME    
Severity: Minor CC: phnixwxz, vincent.bradley
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on:    
Bug Blocks: 41103    
Attachments:
Description Flags
test case none

iain.dalton
Reported 2009-07-12 15:31:55 PDT
Calibre (http://calibre.kovidgoyal.net/) uses Webkit, I'm not sure what version, and it wraps curly quotes (”) to the next line, which looks weird, and not book-like. I'm guessing this is a WebKit bug. I'm on Linux, though, so I don't know how to test this against a WebKit browser.
Attachments
test case (785 bytes, text/html)
2010-10-20 19:36 PDT, Xianzhu Wang
no flags
Xianzhu Wang
Comment 1 2010-10-20 19:36:16 PDT
Created attachment 71383 [details] test case I can't reproduce the bug with the attached test in Linux Chrome 7.0.517.41, Windows Chrome 8.0.552.5, Mac Chrome 7.0.503.0 and Mac Safari 5.0.2. I even can't reproduce in Qt version of WebKit (trunk version). All the browsers break lines neither before or after the quotation mark (Unicode 201D, RIGHT DOUBLE QUOTATION MARK).
Pat
Comment 2 2010-11-08 11:07:40 PST
Using QtWebKit trunk (wk45), this is not reproducible. The test content is using entity &#201D; and a space between some characters. The word break occurs at the space, not around the curly quote. If all spaces are removed the line doesn't break (one long line). Tried with the test content saved as UTF-8 with and without BOM, also UTF-16Used QtTestBrowser to verify on Symbian and Linux. Both work as expected. Test content, as provided, used: <!DOCTYPEhtml> abcd efg&#x201d;abcd efg&#x201d;abcd efg&#x201d;abcd efg&#x201d;abcd efg&#x201d;abcd efg&#x201d; .... Note, that test content has no meta tag: <meta http-equiv="content-type" content="text/html; charset=UTF-16"> Tested with meta tag added and works as expected. The behavior is the same for Firefox, Safari, and QtTestBrowser. QtTestBrowser: /usr/demos/browser/browser.exe
Note You need to log in before you can comment on or make changes to this bug.