1318827 - CKEditor config: Enable spell check by default

Status: RESOLVED INVALID

(developer.mozilla.org Graveyard :: Editing, defect)

Description

[Eric Shepherd [:sheppy]]

This is as simple as adding this line to the ckeditor_config.js file:

config.scayt_autoStartup = true;

Comment 1

[Eric Shepherd [:sheppy]]

As part of this, we should also look at turning on multi-language support for the spell checker:

config.scayt_multiLanguageMode = true;

Comment 2

[Eric Shepherd [:sheppy]]

I have submitted https://github.com/mozilla/kuma/pull/4067 for this.

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/d79e02633563268d1d90aed0086acfb5d4bd0070
bug 1318827 - Disable auto-load of spell checker

We need to look more closely at the licensing situation.

Comment 4

[John Whitlock [:jwhitlock]]

The short version:

Spellcheck is provided by a SCAYT plugin, which is well-integrated with CKEditor.  The CKEditor license allows for free usage, but displays ads to users.

The security concern is the execution of SCAYT code and advertisements inside the MDN context.  We can avoid this by paying a license that removes the ads:

https://www.webspellchecker.net/signup/licensed-signup.html#wsc-trial

We could replace the plugin with browser-native spellcheck. However, this collides with the CKEditor context menu plugin, so we'd have to make hard UX decisions.  Two clear options are:

- Right-click gets the CKEditor context menu, Control + Right-click gets the browser spell check menu
- Remove the CKEditor context menu, Right-click to get the browser spell check

A technical option may also be possible, implementing our own spellcheck and CKEditor plugin. This will be more expensive than the SCAYT license, and we don't have the resources to prioritize this with current development staff.

I'll start a document with the longer version, and a mailing list discussion for team and community feedback.