Adding javascript to let user click table cell to check containing checkbox.
This is a usability improvement as it widens the target of the click. The script for moremember.pl includes code necessary to uncheck its sibling as per uncheck_sibling(). If this patch is approved I will look for other instances so that <td> interactions will be consistent. Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
This commit is contained in:
parent
d7fc49f0a7
commit
36e384adc7
2 changed files with 20 additions and 0 deletions
|
@ -6,6 +6,7 @@
|
|||
<!-- /TMPL_IF --></title>
|
||||
<!-- TMPL_INCLUDE NAME="doc-head-close.inc" -->
|
||||
<script type="text/javascript" src="<!-- TMPL_VAR name="themelang" -->/lib/jquery/plugins/jquery.tablesorter.min.js"></script>
|
||||
<script type="text/javascript" src="<!-- TMPL_VAR name="themelang" -->/lib/jquery/plugins/jquery.checkboxes.pack.js"></script>
|
||||
<script type="text/JavaScript" language="JavaScript">
|
||||
//<![CDATA[
|
||||
$.tablesorter.addParser({
|
||||
|
@ -39,6 +40,13 @@ $.tablesorter.addParser({
|
|||
}
|
||||
} ).attr( 'checked', false );
|
||||
<!-- /TMPL_IF -->
|
||||
$("td").click(function(e){
|
||||
if(e.target.tagName.toLowerCase() == 'td'){
|
||||
$(this).find("input:checkbox").each( function() {
|
||||
$(this).attr('checked', !$(this).attr('checked'));
|
||||
});
|
||||
}
|
||||
});
|
||||
});
|
||||
//]]>
|
||||
</script>
|
||||
|
|
|
@ -51,6 +51,18 @@ $(document).ready(function() {
|
|||
}
|
||||
} ).attr( 'checked', false );
|
||||
<!-- /TMPL_IF -->
|
||||
$("td").click(function(e){
|
||||
if(e.target.tagName.toLowerCase() == 'td'){
|
||||
$(this).find("input:checkbox").each( function() {
|
||||
$(this).attr('checked', !$(this).attr('checked'));
|
||||
if($(this).attr('checked')){
|
||||
$(this).parent().siblings().find("input:checkbox").each(function(){
|
||||
if($(this).attr('checked')){ $(this).attr('checked',''); }
|
||||
});
|
||||
}
|
||||
});
|
||||
}
|
||||
});
|
||||
});
|
||||
function uncheck_sibling(me){
|
||||
nodename=me.getAttribute("name");
|
||||
|
|
Loading…
Reference in a new issue