Bug 118136 - [BlackBerry] We should clear all markers when input changes a word
Summary: [BlackBerry] We should clear all markers when input changes a word
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nima Ghanavatian
Depends on:
Reported: 2013-06-27 07:57 PDT by Nima Ghanavatian
Modified: 2013-06-27 09:33 PDT (History)
1 user (show)

See Also:

Patch (2.48 KB, patch)
2013-06-27 08:02 PDT, Nima Ghanavatian
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nima Ghanavatian 2013-06-27 07:57:40 PDT
For the case where a letter is added to the middle of a misspelled word, we were
sending incorrect offsets to clear spelling markers. However, since this
expanded the current word, it overlaps entirely the previous spelling marker so
no issue is found. However, if the keypress is backspace, the incorrect range
is smaller, causing an overlap which recreates the marker over the last character.
Setting shouldEraseMarkersAfterChangeSelection to follow continuous spell checking,
which will clear all markers regardless of overlap.
This patch also sustains spelling markers after the user taps to move the caret onto
the word, which is a nice gain as it was asked for previously.
Comment 1 Nima Ghanavatian 2013-06-27 08:02:00 PDT
Created attachment 205609 [details]
Comment 2 Rob Buis 2013-06-27 08:20:12 PDT
Comment on attachment 205609 [details]

Looks good.
Comment 3 WebKit Commit Bot 2013-06-27 09:33:24 PDT
Comment on attachment 205609 [details]

Clearing flags on attachment: 205609

Committed r152102: <http://trac.webkit.org/changeset/152102>
Comment 4 WebKit Commit Bot 2013-06-27 09:33:26 PDT
All reviewed patches have been landed.  Closing bug.