- Notifications
You must be signed in to change notification settings - Fork433
Replaced Sortable with ng-sortable#166
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
replaced Sortable with ng-sortable wrapperchanged sample to use ng-sortable
sdorra commentedOct 18, 2015
We had some problems with ng-sortable, during the drag and drop implementation with sortable (in particular with deactivated debug mode). It looks that the debug mode problem is fixed (#255). We have todo some tests and we have to fix the unit tests, but i think it is a great idea to replace the drag and drop code with the ng-sortable directive. |
sdorra commentedOct 18, 2015
I've done some tests with the samples and i got a lot of issues: |
bojanbass commentedOct 18, 2015
Could you tell me which actions caused these errors? Also I forgot to fix unit tests. |
Add dependencies.yml for dependencies.io
Hi,
I made a replacement inside column directive to use ng-sortable angular wrapper for Sortable library. I removed a lot of unused code from column directive this way. Please check, if it makes sense to use this one in your framework. A downside is that we have to include a new js library ng-sortable.js and inject it as angular dependency at startup. I also fixed the sample code to work with these changes.
Regards.