diff options
author | Jared Dillard <jdillard@netgate.com> | 2015-11-25 15:10:39 -0600 |
---|---|---|
committer | Jared Dillard <jdillard@netgate.com> | 2015-11-25 15:10:46 -0600 |
commit | b5ef3ee9902b3168937320e7bb161243b4fad939 (patch) | |
tree | 1a99be8668a5349c187986f65f32f8b00fd0167d /src/usr/local/www/jquery | |
parent | c64e401c85f3ef081a8c6b9e4a8d0a4b4a9835f7 (diff) | |
download | pfsense-b5ef3ee9902b3168937320e7bb161243b4fad939.zip pfsense-b5ef3ee9902b3168937320e7bb161243b4fad939.tar.gz |
clean up firewall rules tables
Diffstat (limited to 'src/usr/local/www/jquery')
-rw-r--r-- | src/usr/local/www/jquery/pfSenseHelpers.js | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/src/usr/local/www/jquery/pfSenseHelpers.js b/src/usr/local/www/jquery/pfSenseHelpers.js index 867d3f1..966e798 100644 --- a/src/usr/local/www/jquery/pfSenseHelpers.js +++ b/src/usr/local/www/jquery/pfSenseHelpers.js @@ -147,11 +147,6 @@ function hideLabel(text, hide) { // Striping of the tables is handled here, NOT with the Bootstrap table-striped class because it would // get confused when rows are sorted or deleted. -function stripe_table() { - $("tr:odd").addClass('active'); - $("tr:even").removeClass('active'); -} - function fr_toggle(id, prefix) { if (!prefix) prefix = 'fr'; @@ -161,8 +156,7 @@ function fr_toggle(id, prefix) { fr_bgcolor(id, prefix); } -// Change background color based on state of checkbox -// On resetting background, reapply table striping +// Change background color of selected row based on state of checkbox function fr_bgcolor(id, prefix) { if (!prefix) prefix = 'fr'; @@ -170,11 +164,9 @@ function fr_bgcolor(id, prefix) { var row = $('#' + prefix + id); if ($('#' + prefix + 'c' + id).prop('checked') ) { - row.css("background-color", "#DDF4FF"); - row.removeClass('active'); + row.addClass('active'); } else { - row.css("background-color", "#FFFFFF"); - stripe_table(); + row.removeClass('active'); } } |