Support for django-admin-sortable2 (feature)

[zypro]

trafficstars

Hi there, The django-admin-sortable2 works pretty well with standard django admin but not with jazzmin. It is used to sort objects directly in the admin table. Jazzmin prohibits the sortableadmin to add some needed ids and css classes to elements like <table>.

Maybe the conflict is caused by overriding change_list.html template by both packages. And it is possible that django-admin-sortable2 has to be updated to bring support in this case.

[farridav]

Usually we are able to provide support for 3rd party packages, even if they override the change list template.

We will need to take a look at this, or if you have the time, feel free to have a stab at it in a PR..

[zypro]

I had a quick look at it today so it might be possible that the 3rd party lib has to adapt.

[farridav]

The fastest way for us to provide a fix in for this, is to do the following:

1. Fork the project and make a new branch
2. Integrate this 3rd party app into our embedded demo app (https://github.com/farridav/django-jazzmin/tree/master/tests/test_app)
3. push it up and send us a link to the branch

This way we will be able to reproduce the issue quickly, and fix it, without needing to delve into how to integrate/setup the 3rd party app :)

Thanks for reporting, we hope to provide compatibility with all other packages, especially if they are widely used

[budlight]

I think on this issue, it should be implemented on the Book -> Author relationship in the test app

Books can have multiple authors and you normally sort them by importance.

Also Admin LTE has sortable built in:

so it is a question of if this package should support django-admin-sortable2 or this build its own sortable attribute and update it with a view.