Bug 9978

Summary:

REGRESSION (r12949-r12988): Clicking the first letter on a line places the caret at the end of the previous line

Product:

WebKit

Reporter:

mitz

Component:

HTML Editing

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

darin, justin.garcia

Priority:

Keywords:

Regression

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

data:text/html,<div%20contenteditable%20style="width:200px;font-size:48">Very%20Wide</div>

Attachments:

Description	Flags
Patch. No test or change log yet.	none
Patch, including test and chance log	justin.garcia: review+

mitz

Reported 2006-07-17 15:13:36 PDT

To reproduce, open the URL and click somewhere in the left half of the W on the second line. The caret should move to the start of the second line, but instead it appears at the end of the first line, that is, the position is correct but the affinity is wrong.

Attachments
Patch. No test or change log yet. (807 bytes, patch) 2006-07-17 15:15 PDT, mitz	no flags	Details Formatted Diff Diff
Patch, including test and chance log (50.19 KB, patch) 2006-07-17 23:20 PDT, mitz	justin.garcia: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

mitz

Comment 1 2006-07-17 15:15:41 PDT

Created attachment 9531 [details] Patch. No test or change log yet.

Darin Adler

Comment 2 2006-07-17 19:42:38 PDT

There's a chance my patch to fix pre and pre-wrap editing fixes this too. It removes some misguided code that could defeat our affinity-handling code path. But it's also possible that it's unrelated.

Darin Adler

Comment 3 2006-07-17 21:54:32 PDT

The patch looks pretty good to me. Justin?

mitz

Comment 4 2006-07-17 23:20:04 PDT

Created attachment 9541 [details] Patch, including test and chance log

David Kilzer (:ddkilzer)

Comment 5 2006-07-18 03:49:41 PDT

Committed revision 15500.

Note You need to log in before you can comment on or make changes to this bug.