Bug 59968 - Text does not split on white space when typing to the edge of window
Summary: Text does not split on white space when typing to the edge of window
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Enrica Casucci
URL:
Keywords: InRadar
: 57745 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-05-02 15:24 PDT by Enrica Casucci
Modified: 2011-06-11 13:25 PDT (History)
5 users (show)

See Also:


Attachments
Patch (8.80 KB, patch)
2011-05-02 15:34 PDT, Enrica Casucci
mitz: review+
Details | Formatted Diff | Diff
RTL testcase (774 bytes, text/html)
2011-05-07 15:06 PDT, Yair Yogev
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Enrica Casucci 2011-05-02 15:24:29 PDT
Repro steps

1. Open the attached test case in Safari.
2. Place the cursor at the end of the text, after the word "you".
3. Type a blank character, then the word "have".

Expected

After typing the blank character the cursor should move to the very end of the line and after typing "have" you should see the word "you" at the end of the first line and the word "have" at the beginning of the second line.

Actual

The word "youhave" (no blanks) is displayed at the beginning of the second line.

Note: this test is platform specific, since it involves testing the condition where the blank character doesn't fit in the previous line by only a fraction of pixel, but the bug occurs on every platform.
Comment 1 Enrica Casucci 2011-05-02 15:34:19 PDT
Created attachment 91993 [details]
Patch
Comment 2 Enrica Casucci 2011-05-02 15:46:43 PDT
http://trac.webkit.org/changeset/85535
Comment 3 Ryosuke Niwa 2011-05-02 15:54:26 PDT
*** Bug 57745 has been marked as a duplicate of this bug. ***
Comment 4 Yair Yogev 2011-05-07 15:06:28 PDT
Created attachment 92697 [details]
RTL testcase

looks like the fix here was only good for LTR text?
see RTL testcase attached
Comment 5 Yair Yogev 2011-05-16 07:54:43 PDT
Comment on attachment 92697 [details]
RTL testcase

posted as bug 60884
Comment 6 Ryosuke Niwa 2011-06-11 13:25:20 PDT
It seems like the test added by this test only works on Mac due to font metrics difference.  Can we move the test to platform/mac or modify it so that it works on all platforms?