Bug 978088 - Spell checking no longer works in Chrome on initial load
Summary: Spell checking no longer works in Chrome on initial load
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: Web-UI
Version: 1.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: Matthew Casperson
QA Contact:
URL:
Whiteboard:
: 979196 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-26 00:30 UTC by Lee Newson
Modified: 2014-08-04 22:27 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-09 00:00:38 UTC
Embargoed:


Attachments (Terms of Use)

Description Lee Newson 2013-06-26 00:30:57 UTC
Since the change to using web workers for the spell checking, it no longer works in Chrome and fails with the following error:

Uncaught TypeError: Cannot call method 'check' of undefined :8080/pressgang-ccms-ui-next/javascript/typojs/checkspelling.js:179

Comment 1 Lee Newson 2013-06-26 00:36:20 UTC
Just found another interesting part, it only appears to fail when the page is initially loaded. If you do a search or go to a topic directly and then click the topic in the search results pane, the spell checking will start to work.

Comment 2 Lee Newson 2013-06-27 22:50:32 UTC
*** Bug 979196 has been marked as a duplicate of this bug. ***

Comment 3 Matthew Casperson 2013-07-01 01:21:23 UTC
The issue appears to be because the editor is only waiting for the positive dictionary to load, and is not waiting for the negative or negative phrase dictionaries to load.

Comment 4 Matthew Casperson 2013-07-01 01:25:48 UTC
Fixed in 201307011121

NOTE:
When QAing bug fixes, please make sure the build you are using is equal to or higher than the build identified above. Be aware that this fix may have only been applied in PressGang Next at this time, or it may not be currently applied in any production system, in which case it can only be verified at a later date.

Comment 5 Lee Newson 2013-07-01 02:45:28 UTC
The issue still remains in Build 201307011121. This is testing in Incognito mode and completely clearing my cache.

Also changing the title to be more descriptive.

Comment 6 Lee Newson 2013-07-01 03:19:38 UTC
Just adding additional details.

Now the spell checking fails earlier by the looks of it. The error is now:

Uncaught ReferenceError: console is not defined :8080/pressgang-ccms-ui/javascript/typojs/checkspelling.js:41

By the looks of it the spell checking is running in a web worker, in which case it doesn't have access to the console object (see: http://stackoverflow.com/questions/7251779/web-worker-console-log or http://www.htmlgoodies.com/html5/client/introduction-to-html5-web-workers-browser-support-non-accessible-elements-and-error-handling.html)

Comment 7 Lee Newson 2013-07-01 03:48:25 UTC
Forgot to mention, that the main problem is that the dictionary is still null.

Comment 8 Matthew Casperson 2013-07-17 00:39:29 UTC
Fixed in 201307171035

This was an issue with the sequence in which the dictionary data was passed to the worker.

Comment 9 Lee Newson 2013-07-17 04:04:40 UTC
Verified, so setting this to RELEASE_PENDING.


Note You need to log in before you can comment on or make changes to this bug.