[Webkit-unassigned] [Bug 65898] DeleteWordForward deletes 2 words
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon May 28 18:34:21 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=65898
Antaryami Pandia (apandia) <antaryami.pandia at motorola.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |antaryami.pandia at motorola.c
| |om
--- Comment #9 from Antaryami Pandia (apandia) <antaryami.pandia at motorola.com> 2012-05-28 18:34:20 PST ---
I have checked the issue with some other words and below is the observation:
1: foo bar -baz_ --> control+backspace --> foo bar -
2: foo bar @baz_ --> control+backspace --> foo bar @
3: foo bar _baz_ --> control+backspace --> foo
The below code in method "findNextWordFromIndex" is used to calculate the position of character after ctrl+bkspace:-
position = textBreakPreceding(it, position);
case 1 and 2
============
returns the position text just after the special character ie (b) whose position is 9. So the isAlphanumeric check passes and returns the same position.
case 3
======
The position returned is 8, i.e position of (_) . So the isAlphanumeric check fails and it continues to find the next position. And it returns the position of 4 which is the start of 2nd word. So it deletes upto that point.
The "findNextWordFromIndex" method uses the ICU library to determine the position. So I think the ICU library should also returns the correct position when underscore(_) is used.
Also what should be the right output:
foo bar _baz_ --> control+backspace --> foo bar
OR
foo bar _baz_ --> control+backspace --> foo bar _
Please provide your feedback.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list