search

badsyntax / jquery-spellchecker

[not maintained] A lightweight jQuery plugin that can check the spelling of text within a form field or DOM tree.
http://jquery-spellchecker.badsyntax.co
MIT License
257 stars 113 forks source link

When spellchecking with IE8, extra paragraphs are created occasionally #35

Open peter-klaesson-stratsys opened 11 years ago

peter-klaesson-stratsys commented 11 years ago

To reproduce: (use native IE8, not IE9 in IE8 mode)

Go to http://jquery-spellchecker.badsyntax.co/ckeditor.html

Place the cursor between "great explorer" and "of the truth" in the paragraph. Hit enter.

Press spellchecker (ABC-button on the top of the editor).

Spellcheck the second paragraph (starting with 'of the truth'... Correct Noo, consequencses's and painful.

At either "consequencses's" or "painful", the normalize function will create an extra paragraph when normalizing the p-element.

Created a pull request at https://github.com/padolsey/findAndReplaceDOMText/pull/9 that will fix this problem, hopefully.

alang commented 11 years ago

The word "painful" is spelled correctly and shouldn't be highlighted to begin with. Perhaps this is linked to issues in #26?

peter-klaesson-stratsys commented 11 years ago

No, #26 is not related to this issue. This issue only affects IE8 and is solved by deferring the text normalization. (It is a work around for some timing issue in IE8)