Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 314453 - Disable spell checking in Chrome
Summary: Disable spell checking in Chrome
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.4 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: sr131
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-26 09:01 EDT by Rüdiger Herrmann CLA
Modified: 2010-08-31 11:26 EDT (History)
0 users

See Also:


Attachments
Screenshot (50.70 KB, image/png)
2010-05-26 09:02 EDT, Rüdiger Herrmann CLA
no flags Details
Fix (772 bytes, patch)
2010-05-27 15:16 EDT, Rüdiger Herrmann CLA
no flags Details | Diff
Fix (2.57 KB, patch)
2010-06-30 16:04 EDT, Rüdiger Herrmann CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rüdiger Herrmann CLA 2010-05-26 09:01:17 EDT
Chrome >= 5.0 has spell checking enabled for input widgets. Unrecognized words are underlined in red.
Please disable spell checking in Chrome for all input widgets.
Comment 1 Rüdiger Herrmann CLA 2010-05-26 09:02:27 EDT
Created attachment 169980 [details]
Screenshot
Comment 2 Ralf Sternberg CLA 2010-05-27 06:50:44 EDT
There's a spellcheck attribute in HTML5:
http://dev.w3.org/html5/spec/Overview.html#spelling-and-grammar-checking

Google chrome seems to support this attribute but ignores it if set from Javascript (http://code.google.com/p/chromium/issues/detail?id=25060). Would it help to render it as html (if we don't do so already)?
Comment 3 Rüdiger Herrmann CLA 2010-05-27 15:16:20 EDT
Created attachment 170246 [details]
Fix

This fixes the issue by adding a spellcheck="false" attribute to the body tag of the rwt-index.html.
One strange thing I noticed is that Text.js sems to turn spellchecking on (!) in its constructor
Comment 4 Rüdiger Herrmann CLA 2010-06-30 16:04:39 EDT
Created attachment 173153 [details]
Fix
Comment 5 Rüdiger Herrmann CLA 2010-06-30 16:05:43 EDT
Fixed in CVS HEAD
Comment 6 Ralf Sternberg CLA 2010-08-31 11:26:03 EDT
Applied patch to 1.3 maintenance branch