[Webkit-unassigned] [Bug 48078] New: Editing a word with misspell or autocorrection underline should remove the underline when the editing changes the word.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=48078

           Summary: Editing a word with misspell or autocorrection
                    underline should remove the underline when the editing
                    changes the word.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Macintosh
        OS/Version: Mac OS X 10.6
            Status: NEW
          Keywords: InRadar
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jpu at apple.com
            Blocks: 47630


After bug 47629, on Mac OS 10.7, when moving cursor onto a word, the underline (if the word has) is not removed. However, if now the user start editing the word, then we need to move the underline first before applying the underline. If after the editing, the word is still misspelled, we will rely on spell checking to add the underline back on.

Note, this is a 10.7 only change. 


At this moment, say we have a word "mispelling" with red misspelling underline. Now if we insert "s" to change it to "misspelling", the result is "mis" and "pelling" with red underline, but not "s". Also, noticed that the word is now correctly spelled. Similar behavior happens to blue autocorrection underline. 

To fix this, if an editing command changes a word that has misspell or autocorrection underline, we need to remove those underlines before applying the editing.

There're 3 scenarios to consider:
1. If the editing happens in the middle of the word. We always remove the underline.
2. If the editing happens at either end of the word, and the editing is to inserting a whitespace, we don't need to remove the underline, because this will not change the word.
3. If the editing happens at either end of the word, and the editing is not simply inserting a whitespace, we remove the underline.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.