Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

$.fn.txpMultiEditForm to on() from deprecated live() method, fixes caching issues [patch] #246

Closed
ghost opened this issue Oct 4, 2014 · 2 comments

Comments

@ghost
Copy link

@ghost ghost commented Oct 4, 2014

From jukka.svahn@rahinaa.biz on August 09, 2012 11:25:37

Please see the attached patch. It updates the txpMultiEditForm() to use .on() method too.

This patch also fixes certain caching issues related to checkboxes. Previously the private checked() method used cached values for checking whether all rows were selected and deciding the select_all[] box's status. Using cached nodes led to potential temporary errors (lasting for the first check) when a table contained dynamically added rows.

Attachment: textpattern.js.r4056.patch

Original issue: http://code.google.com/p/textpattern/issues/detail?id=242

@ghost ghost added the imported label Oct 4, 2014
@ghost
Copy link
Author

@ghost ghost commented Oct 4, 2014

From r.wetzlmayr on August 09, 2012 03:03:51

Status: Assigned
Owner: r.wetzlmayr

Loading

@ghost
Copy link
Author

@ghost ghost commented Oct 4, 2014

From r.wetzlmayr on August 09, 2012 03:08:23

This issue was closed by revision r4057 .

Status: Fixed

Loading

@ghost ghost closed this Oct 4, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
0 participants