Summary: | [chromium] Spellchecker should show suggestions only when right-clicking a misspelled word. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Hironori Bono <hbono> | ||||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | morrita, progame+wk, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | 89444 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Hironori Bono
2012-06-17 23:53:35 PDT
Created attachment 148065 [details]
Patch v1
Greetings Morita-san,
Would it be possible to review this quick fix?
Regards,
Hironori Bono
Comment on attachment 148065 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=148065&action=review The code looks good. > Source/WebKit/chromium/ChangeLog:4 > + https://bugs.webkit.org/show_bug.cgi?id=89331 Could you give a bit more detailed explanation? Seeing only this change doesn't give any rationale behind the change. Created attachment 148070 [details]
Patch v1 (updated ChangeLog)
Greetings Morita-san,
Many thanks for your quick review.
I have updated ChangeLog to describe the background.
Regards,
Hironori Bono
Comment on attachment 148070 [details] Patch v1 (updated ChangeLog) Rejecting attachment 148070 [details] from review queue. hbono@chromium.org does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py. - If you do not have reviewer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags. - If you have reviewer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed). The commit-queue restarts itself every 2 hours. After restart the commit-queue will correctly respect your reviewer rights. Comment on attachment 148070 [details] Patch v1 (updated ChangeLog) Clearing flags on attachment: 148070 Committed r120586: <http://trac.webkit.org/changeset/120586> All reviewed patches have been landed. Closing bug. Greetings, Sorry, I noticed this was not a good fix. (It is better for us to emulate Firefox.) Would it be possible to roll out my r120586? Regards, Hironori Bono Re-opened since this is blocked by 89444 Created attachment 148305 [details]
Patch v2 (fixed a regression on Mac)
Greetings Morita-san,
Thanks for your help and apologies for my previous change. I have re-implemented my change to emulate the behavior of Mac Chrome so we do not need workarounds for Mac. Would it be possible to review this change?
Regards,
Hironori Bono
Comment on attachment 148305 [details] Patch v2 (fixed a regression on Mac) Clearing flags on attachment: 148305 Committed r120810: <http://trac.webkit.org/changeset/120810> All reviewed patches have been landed. Closing bug. This fix is the cause of a very weird regression in Windows (only tested windows): let's say i'm posting a comment IGNORETHISWORD in WebKit Bugzilla, just like i am right now . All my words are spelled correctly except for one, that capital letters gibberish in the second line. Now i would like to copy my comment before submitting, so i: 1. Press Ctrl+A 2. right click the selection (anywhere) The selection will now change: The part of the text before the gibberish word will be deselected and if the last text block is longer than the horizontal position of the gibberish word, then any text in that block after that position will also be deselected. screenshot of comment #12 http://i47.tinypic.com/2myrzpf.jpg Was this change suppose to actually change the selection? i don't think it should... For one lined text field, with one misspelled word, it deselects the whole selection except for the misspelled word, although i didn't right click the misspelled word and all i wanted to do is Copy the text using the context menu option... I hope that's not intentional, the spell checker shouldn't interfere with basic text handling capabilities. |