Bug 19443 - REGRESSION (r32531-r32652): Moving the insertion point past a newline inserted in a text area skips the next character
Summary: REGRESSION (r32531-r32652): Moving the insertion point past a newline inserte...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: Nobody
URL:
Keywords: InRadar, Regression
Depends on:
Blocks:
 
Reported: 2008-06-08 20:47 PDT by mitz
Modified: 2008-06-09 21:59 PDT (History)
0 users

See Also:


Attachments
Patch (7.76 KB, patch)
2008-06-09 21:01 PDT, mitz
justin.garcia: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2008-06-08 20:47:57 PDT
Steps to reproduce:
1. In a text area, such as on this page, type "foo", press return, and type "bar" on the next line.
2. Place the insertion point on the first line, after "foo".
3. Press the right arrow key.

Expected results:
The insertion point should move to the next line, just before the "b".

Actual results:
The insertion point moved to the next line, between "b" and "ar".

Regression:
bisect-builds puts this in the r32531-r32652 range. The visual arrow keys (for bidi text) patch was landed in that range.

Note:
This does not seem to happen with a content-editable div or with a text area already containing the line break.
Comment 1 mitz 2008-06-08 20:48:33 PDT
<rdar://problem/5994544>
Comment 2 mitz 2008-06-08 20:50:48 PDT
(In reply to comment #0)
> The visual arrow keys (for
> bidi text) patch was landed in that range.

<http://trac.webkit.org/changeset/32605>.
Comment 3 mitz 2008-06-09 21:01:58 PDT
Created attachment 21599 [details]
Patch
Comment 4 Justin Garcia 2008-06-09 21:54:09 PDT
Comment on attachment 21599 [details]
Patch

r=me
Comment 5 mitz 2008-06-09 21:59:54 PDT
Fixed in <http://trac.webkit.org/changeset/34475>.