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

Redactor demo IE7/IE8 #19

Open carlospictor opened 11 years ago

carlospictor commented 11 years ago

Hi,

I've just been looking at the redactor demo in IE7 and IE8 and have noticed that the response from calling SpellChecker.php always comes back empty - with no word suggestions.

Any thoughts as to why this might be happening? the text parameter is sent as text[] - is IE <9 unable to pass this correctly perhaps?

badsyntax commented 11 years ago

Thanks for this bug report. I've just done some testing and this seems to be a general ie8/7 bug with the spellchecker. Not sure where this regression happened :( I will try get this fixed over the weekend.

carlospictor commented 11 years ago

Hi,

Have done a bit more dbugging, and in IE7/8, it looks as though the text[] parameter is being split on every character: text%5B%5D=T+h+s+i+i+s+s+p+e+l+l+l+e+d+b+a+d+l+y+l

As opposed to: text%5B%5D=thsi+is+spelled+badlyly

Am not sure where the splitting occurs, but it looks the text is being split too much in IE7/8?

badsyntax commented 11 years ago

I've done a lot of fixes which should hopefully resolve this issue. Please can you test with the latest version of the plugin? You can also test here: http://jquery-spellchecker.badsyntax.co/redactor.html (clear your cache before testing though).

carlospictor commented 11 years ago

Hi,

I've tested on your local example and also updated my dev example, and the spellcheck now works great - thank you. the only thing I have spotted is that if the very first word is misspelled, it doesn't get picked up in any browser. Adding a space in front of the word does then let it get checked.