RESOLVED FIXED Bug 35066
keyboard selection sometimes moves the wrong end of the selection for Win/Linux
https://bugs.webkit.org/show_bug.cgi?id=35066
Summary keyboard selection sometimes moves the wrong end of the selection for Win/Linux
Ojan Vafai
Reported 2010-02-17 18:19:59 PST
Given the following text: line one line two line three 1. Put the cursor in the middle of line one. 2. Shift+down 3. Shift+home 4. Shift+end On Win/Linux, you should see the word "one" selected after 3. And the words "one" and "line two" selected after 4. Instead "line " (in line 1) is selected after 3 and "one" is selected after 4.
Attachments
Patch (65.18 KB, patch)
2010-02-17 18:24 PST, Ojan Vafai
abarth: review+
Ojan Vafai
Comment 1 2010-02-17 18:24:11 PST
Adam Barth
Comment 2 2010-02-18 00:54:13 PST
Comment on attachment 48957 [details] Patch I applaud your thorough testing. Everyone seems really excited about this patch and the mechanics seem straightforward enough. I'm not an editing expert, so please let me know if you'd rather I didn't review editing patches in the future.
Evan Martin
Comment 3 2010-02-18 00:59:00 PST
+// FIXME: Expose settings->editingBehavior() and use that here instead. +var onMacPlatform = navigator.userAgent.search(/\bMac OS X\b/) != -1; I wonder if this will fail on iPhone. I wonder if iPhones use Mac editing behavior. I guess there's no shift-home on iPhones. :) I wonder if we run tests on iPhones?
Tony Chang
Comment 4 2010-02-18 01:11:03 PST
(In reply to comment #3) > +// FIXME: Expose settings->editingBehavior() and use that here instead. > +var onMacPlatform = navigator.userAgent.search(/\bMac OS X\b/) != -1; > > I wonder if this will fail on iPhone. > I wonder if iPhones use Mac editing behavior. > I guess there's no shift-home on iPhones. :) > I wonder if we run tests on iPhones? The iPhone user agent has Mac OS X in it so this should be fine. This would matter for an iPad with a keyboard.
Ojan Vafai
Comment 5 2010-02-18 11:38:22 PST
Note You need to log in before you can comment on or make changes to this bug.