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

newIndex incorrect behaviour #447

Closed
avalanche1 opened this issue Jun 24, 2015 · 9 comments
Closed

newIndex incorrect behaviour #447

avalanche1 opened this issue Jun 24, 2015 · 9 comments
Labels

Comments

@avalanche1
Copy link

newIndex randomly returns incorrect values if the dragged item's position hasn't been changed, like picked up, moved around, but dropped into the old position. This begins to take place after several drag\drops.
http://jsbin.com/fuxigukuja/1/edit?js,console,output

@RubaXa
Copy link
Collaborator

RubaXa commented Sep 9, 2015

Not reproduced.

@RubaXa RubaXa closed this as completed Sep 9, 2015
@avalanche1
Copy link
Author

How is this not reproduced?? did you check my bin?
image

@RubaXa
Copy link
Collaborator

RubaXa commented Sep 15, 2015

@avalanche1
Copy link
Author

cannot confirm - strangely my jsbin, as well as yours, don't produce absolutely no input to console...

@RubaXa
Copy link
Collaborator

RubaXa commented Sep 18, 2015

Strangely, works fine: http://recordit.co/2Vp7T8t8UJ

@avalanche1
Copy link
Author

ah, yes! works fine! thanks for fixing)

@RubaXa RubaXa closed this as completed Sep 18, 2015
RubaXa added a commit that referenced this issue Sep 22, 2015
@avalanche1
Copy link
Author

Could you push latest release to Atmosphere plz.

@RubaXa
Copy link
Collaborator

RubaXa commented Sep 24, 2015

I already wrote to @dandv, that the publication does not work, but he did not answer.

grunt meteor
Running "exec:meteor-test" (exec) task
### Testing rubaxa:sortable...
>> meteor/runtests.sh: line 31: ./node_modules/.bin/spacejam: No such file or directory
>> Exited with code: 127.

If you know how to fix it, I'll be happy to accept PR.

@avalanche1
Copy link
Author

Maybe that's related to new package dependency system introduced in Meteor 1.2?
Let's wait for @dandv as he is surely knowledgeable in this.

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

No branches or pull requests

2 participants