search

darklow / django-suit

Modern theme for Django admin interface
http://djangosuit.com/
Other
2.31k stars 701 forks source link

Change form.onsubmit to eventlistener #666

Open iamkhush opened 6 years ago

iamkhush commented 6 years ago

In our usecase, we have another library that overloads form.onsubmit, and submits the form. However, the formIsDirty is also true ( because of the changes to Django Multiple Select Box ) and hence the confirmation pops up. In the proposed change, Django suit would work together nicely with other libraries.

codingjoe commented 6 years ago

@iamkhush wouldn't it be also a good idea to make the same change on django-s3file?

codingjoe commented 6 years ago

@iamkhush fyi https://github.com/codingjoe/django-s3file/pull/80