Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

'destroy' option #15

Open
pierresh opened this issue Oct 1, 2014 · 0 comments
Open

'destroy' option #15

pierresh opened this issue Oct 1, 2014 · 0 comments

Comments

@pierresh
Copy link

pierresh commented Oct 1, 2014

Hello,

I think this plugin is great, fully in lines with KISS principles! Nevertheless I am using it on pages whose table can be refreshed many times, have lines added dynamically, and also be validated. Consequently I would be interested to have a 'destroy' option included in this plugin, in order to prevent memory leaks and modifications once the table has been validated. I tried to do by myself but I have not managed to make it working (the table is still editable).

At line 56, I have inserted the following lines:

if(options == 'destroy') { element.find("td").off('click keypress dblclick', showEditor); } else { ... }

Could you please let me know what is wrong?

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant