zorab47 / active_admin-sortable_tree

Show ActiveAdmin index as a nested tree with drag'n'drop
MIT License
162 stars 127 forks source link

needs to be updated for jquery-ui-rails 5.0 #37

Closed igorbernstein closed 9 years ago

igorbernstein commented 10 years ago

activeadmin upgraded to jquery-ui-rails 5.0 in https://github.com/gregbell/active_admin/commit/7b9ad75a93f5edaacdfef09cbf5e1d999826a952

The change should be as simple as: Rename jquery.ui..{js,css} to jquery-ui/.{js,css}

If you aren't ready to track master activeadmin, please add < 5.0 jquery-ui-rails version constraint.

Thanks!

igorbernstein commented 10 years ago

for the time being, I've worked around the issue by adding

app/assets/javascripts/jquery.ui.sortable.js: //= require jquery-ui/sortable

dchen-drizly commented 10 years ago

Thanks Igor that worked for me

cpence commented 10 years ago

+1 on the bug, and thanks for the workaround!

zorab47 commented 9 years ago

Everything is fixed for the new jQuery structure.