search

DataTables / Responsive

Responsive extension for DataTables, providing support for complex tables on all device screen sizes
Other
148 stars 86 forks source link

Destroy instead of hide modal? #84

Closed kevinmickey closed 8 years ago

kevinmickey commented 8 years ago

Using bootstrap, and a modal for the details view, it looks like a new modal is created, and then hidden, each time a details view is requested. This could leave a lot of inactive DOM elements. It seems like it would be better either to destroy instead of hiding the modal, or to check for an existing modal and replace its content. (Note that bootstrap allows only one modal at a time.)

DataTables commented 8 years ago

That makes a lot of sense - thanks! I'll make the required changes for the next release.

DataTables commented 8 years ago

Fixed in 3262ee9. Will be int he next release :-)