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

Jump to bottom

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

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

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

ghost opened this issue Oct 4, 2014 · 2 comments

Comments

@ghost
Copy link

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 commented Oct 4, 2014

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

Status: Assigned
Owner: r.wetzlmayr

@ghost
Copy link
Author

ghost commented Oct 4, 2014

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

This issue was closed by revision r4057 .

Status: Fixed

@ghost ghost closed this as completed 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
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
0 participants