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

Already on GitHub? Sign in to your account

onComplete returns the updated li instead of the ul #5

Merged
merged 2 commits into from Jun 14, 2013

Conversation

Projects
None yet
3 participants
Contributor

andreu86 commented Jun 14, 2013

I think it is more flexible because you can find easily the parent list.

It fixes #4

Contributor

ashishpuliyel commented Jun 14, 2013

Thanks for your contributions. If you also update the minified version, it makes it easier for us to merge the pull requests. We're using the closure complier: http://closure-compiler.appspot.com/home

Thanks again!

Contributor

andreu86 commented Jun 14, 2013

Ok, no problem, any configuration or the default one?

Contributor

ashishpuliyel commented Jun 14, 2013

Just the default - @compilation_level SIMPLE_OPTIMIZATIONS
I should include this in our readme, but you're the first one to contribute!

Contributor

andreu86 commented Jun 14, 2013

Done! You will have to update the readme due to this change.

This plugin is extremely simple and useful, congratulations! it is the best sortable plugin I've found, incredibly lightweight, mouse and touch, etc. Thanks!

@jieyanhuang jieyanhuang merged commit 051b641 into docubuzz:master Jun 14, 2013

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