Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autocomplete affects more than one cell #1021

Closed
ss-proto opened this issue Sep 3, 2013 · 10 comments
Closed

Autocomplete affects more than one cell #1021

ss-proto opened this issue Sep 3, 2013 · 10 comments

Comments

@ss-proto
Copy link

ss-proto commented Sep 3, 2013

Hey there, thanks for this great library. I really enjoy using it and it gives me some great advantages in my project.

I might have found a possible bug - it can be repoduced by following these steps:

  1. go to http://handsontable.com/demo/autocomplete.html
  2. In any autocomplete-cell (e.g. column 1, row 1): click the dropdown-triangle and select a value out of the list
  3. take another autocomplete-cell (e.g. column 1, row 2), click the dropdown-triangle and select another value (see first picture)
  4. by clicking (!) in any other cell, the value (which has been just selected) gets copied into all previously edited autocomplete-cells (see second picture)

2013-09-03 15_16_35-autocomplete cell type - handsontable

2013-09-03 15_17_18-autocomplete cell type - handsontable

Could anybody reproduce this?

Note: if you avoid clicking in another cell (step 4) and use instead the arrow keys, it works fine.

@gravey16
Copy link

gravey16 commented Sep 3, 2013

Experiencing this bug.

@henderjr
Copy link

henderjr commented Sep 3, 2013

+1. Was just going to update from .14 to this .16 and on my first test I experienced this issue. So going to pull back and wait for a fix

@Hokwang
Copy link

Hokwang commented Sep 4, 2013

experience, too

@Hokwang
Copy link

Hokwang commented Sep 4, 2013

and if there is more than 5 autocomplete, can not select any value and autocomplete cells are changed to null

@psmolenski
Copy link
Contributor

Thanks for the report. My fix for #1011 deals with this bug as well. It will be released in version 0.9.17 very soon.

@ghost ghost assigned psmolenski Sep 4, 2013
@sapreuss
Copy link

sapreuss commented Sep 4, 2013

Hi, I experienced this bug and the freeze bug as well.
For me it looks as if the table onCellMouseDown event will be restored after unbinding the Editor. This leead reproducable to the recursion

@psmolenski
Copy link
Contributor

Yes you are right. The problem was caused by the error in onCellMouseDown() function. As I said, I've managed to fix it in this commit: da70461

psmolenski added a commit that referenced this issue Sep 4, 2013
@mariotacke
Copy link

Can't wait for this fix! Thanks so much @psmolenski

@psmolenski
Copy link
Contributor

Fixed in version 0.9.17. Please, upgrade and confirm.

@ss-proto
Copy link
Author

ss-proto commented Sep 6, 2013

Works fine now, thanks for fixing :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants