search

deesmateen / jquery-spellchecker

Automatically exported from code.google.com/p/jquery-spellchecker
0 stars 0 forks source link

Spellchecker breaks when used more than once #2

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. Type (or copy and paste): "I really ace at speelling. "
2. click spellcheck. Correct the spelling of "speeling".
3. Type (or copy and paste): "Oh yes I uam."
4. click spellcheck. Spellchecker will not show list of correct spellings
for "uam".

What is the expected output? What do you see instead?
You should see the drop-down list with suggestion corrections for the
spelling of "uam", no drop-down appears.

What version of the product are you using? On what operating system?
I'm using the most recent version of the spellchecker. This error happens
on both Firefox and IE8 on Windows Vista (other browsers haven't been
tested yet).

Please provide any additional information below.
The error appears to be JS based as you can see the returns from the
webservice call for alternative spellings of "uam".

From viewing the code the issue appears to be in the buildSuggestBox
function (L171). The for loop which adds the links is reached however the
links are never seen in the browser.

Original issue reported on code.google.com by petegrah...@gmail.com on 3 Feb 2010 at 2:26

GoogleCodeExporter commented 8 years ago 
Thank you for this bug report. Unfortunately I have been unable to replicate 
this
issue using the example demo here:
http://spellchecker.jquery.badsyntax.co.uk/example.html

However I have noticed a different issue, and it might be related to the bug 
you've
posted. It seems if you don't correct the word "speeling" (step 2), and paste in
different content, and click "check spelling" again, it won't re-build the 
incorrect
word list. 

I will try get to fixing this asap.

Original comment by willis...@gmail.com on 3 Feb 2010 at 2:35

GoogleCodeExporter commented 8 years ago 
I think i've fixed this. Please can you confirm this by using the latest 
revision.

View change here: 
http://code.google.com/p/jquery-spellchecker/source/detail?r=96

Original comment by willis...@gmail.com on 10 Feb 2010 at 11:52

GoogleCodeExporter commented 8 years ago

Original comment by willis...@gmail.com on 10 Feb 2010 at 11:52

GoogleCodeExporter commented 8 years ago 
Hi willis,

The demo at http://spellchecker.jquery.badsyntax.co.uk/example.html does not 
replace
the words anymore. I've tried in Firefox 3.5.8 on Mac and Safari 4.0.4 on mac. 
I put
the word 'herez' and did spell check. The suggestions came back, but clicking 
one
just caused the box of misspelled words to disappear.

Original comment by sald...@gmail.com on 22 Feb 2010 at 5:03

GoogleCodeExporter commented 8 years ago 
Re: Comment 4

Also, no javascript errors are generated in my firefox console when this 
happens.

Original comment by sald...@gmail.com on 22 Feb 2010 at 5:04

GoogleCodeExporter commented 8 years ago 
hi, thanks for letting me know about this.
i have replicated your issue and will roll a fix out shortly.

please note i have been doing certain upgrades recently and the changes are very
alpha at this time. i am prepping for an initial beta release so that people 
won't
have to keep using the latest (unstable) revision

thanks again

Original comment by willis...@gmail.com on 22 Feb 2010 at 5:13

GoogleCodeExporter commented 8 years ago 
this is fixed with r135

Original comment by willis...@gmail.com on 23 Feb 2010 at 9:28

GoogleCodeExporter commented 8 years ago

Original comment by willis...@gmail.com on 23 Feb 2010 at 9:29