Destroy now calls plugin reset. Destroy also properly deallocates resour... #125

Scottmitch · 2014-05-20T18:19:16Z

...ces if destroyed while editing

Address issue #124

…ources if destroyed while editing

Scottmitch · 2014-05-20T18:21:14Z

The test case has been updated to demonstrate these modifications.

…ee elements when clean up was attempted.

Scottmitch · 2014-05-20T19:16:11Z

Test case updated to reflect last commit's changes.

…Corrected 'this' to 'self' where applicable.

Scottmitch · 2014-05-20T22:02:09Z

Test case updated. Plugins now have destroy() hook. reset() is still called by destroy if jeditable is active to clean up controls.

Scottmitch · 2014-07-31T02:48:45Z

@tuupola - Any updates on this being merged into the release?

NicolasCARPi · 2018-02-15T00:54:00Z

Hello,

Thank you for this very nice contribution. Your code has been merged! :)

Cheers,
~Nico

Scottmitch added 2 commits May 20, 2014 14:05

Destroy now calls plugin reset. Destroy also properly deallocates res…

141a58b

…ources if destroyed while editing

Clean up of timeouts array in the event of a normal timeout fire.

b437a8f

Removed selector style that uses caching and lead to Detatched DOM Tr…

0d6c604

…ee elements when clean up was attempted.

Fixed timer array not be tracked appropriatly for multiple controls. …

3e49d74

…Corrected 'this' to 'self' where applicable.

NicolasCARPi added a commit that referenced this pull request Feb 15, 2018

better destroy. contrib by @Scottmitch. fix #125

8c4b32d

NicolasCARPi closed this Feb 15, 2018

Scottmitch deleted the issue_124 branch February 15, 2018 01:03

Provide feedback