Opened 14 years ago
Closed 14 years ago
#15051 closed enhancement (fixed)
Don't let all <thead> checkboxes to function as selector
Reported by: | Cimmo | Owned by: | |
---|---|---|---|
Milestone: | 3.1 | Priority: | normal |
Severity: | normal | Version: | 3.0.1 |
Component: | JavaScript | Keywords: | has-patch |
Focuses: | Cc: |
Description
In my plugin: Cimy User Extra Fields I use checkboxes on <thead> but unclukily one function in common.js is picking up any click coming from any checkbox from <thead> and <tfoot>
Since I think this is too much, we can maybe take only the one from '.check-column' if this doesn't break anything else.
Here is my patch, please review it and let me know.
Attachments (1)
Change History (6)
#1
@
14 years ago
- Keywords has-patch added
You can condense that into .find('.check-column:checkbox')
.
#3
follow-up:
↓ 4
@
14 years ago
I never used JQuery in my life, I just used my common sense and it fixed the issue. The condensed version looks even better if it works the same, but yes the point is, please apply the patch if you like it.
Note: See
TracTickets for help on using
tickets.
Patch to solve this issue, to be QAed for regressions