I have not looked at the code, and probably wouldn't understand much of it if I did...but I was just messing around with the demo. I love this functionality, but thought I'd point out a small flaw. If I misspell a word, and I choose to "edit" it, after resuming, the edited word is NOT rechecked for errors. So any words that I "edit" that are incorrect will be skipped. The user would have to then manually re-run the "check spelling" function.

I believe that any word that is manually edited should be rechecked after the "resume checking" option is selected.

Comments

Thox’s picture

Thox commented

The purpose of the "edit" is to type in words that the spell checker does not recognise - although I suppose it is also used for words which were spelt very badly and the correct suggestion was simply not offered.

I can't really justify making another server hit simply to recheck a handful of words.

varunvnair’s picture

varunvnair commented

@shane: Your observation is correct but I think there is a reason why it works this way. This spellcheck module (and probably other AJAX based spellcheck modules) are supposed to be lightweight spelling checking tools whose aim is to find most of the misspelled words. Words that have been spelled incorrectly because you made a typing mistake and not because you do not know the correct spelling.

There is no custom dictionary to which you can add your own words (most proper nouns such as names, slang words, jargon words etc.). When the module allows you to manually correct a misspelled word it assumes that the word that you type in is correct for you. Regarding further misspellings it acts like an ostrich and simply ignores them.

@Thox: Maybe you can allow the further spell check of manually corrected misspellings as an admin-configurable option. So people who want that feature can enable that.