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

Add element at random location #3

Merged
merged 1 commit into from Nov 1, 2013
Merged

Conversation

p4bl1t0
Copy link
Contributor

@p4bl1t0 p4bl1t0 commented Oct 31, 2013

Add a random index to insert the new element in order to see differents magicmoves animation

Add a random index to insert the new element in order to see differents magicmoves animation
maccman added a commit that referenced this pull request Nov 1, 2013
Add element at random location
@maccman maccman merged commit 4bc5049 into maccman:master Nov 1, 2013
@maccman
Copy link
Owner

maccman commented Nov 1, 2013

Good stuff!

maccman added a commit that referenced this pull request Nov 1, 2013
This reverts commit 4bc5049, reversing
changes made to 20d475a.
@maccman
Copy link
Owner

maccman commented Nov 1, 2013

Oops, this isn't working for me. Reverting till I find out why.

@p4bl1t0
Copy link
Contributor Author

p4bl1t0 commented Nov 2, 2013

I tested it in lastest Chrome and lastest Firefox and works fine for me.

What problem do you experiment?

I couldn't tested on IE 9 because I am on a Linux platform

@p4bl1t0
Copy link
Contributor Author

p4bl1t0 commented Nov 4, 2013

More testing at http://p4bl1t0.github.io/jquery.magicmove/index.html
IE 10 : Works well
IE9: No animation and in original neither
Opera presto : Works well
Opera Webkit: Works well
Safari windows: Works well

Couldn't find where the issue is ...

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

Successfully merging this pull request may close these issues.

None yet

2 participants