Reengineer Form_Button setAttribute

As per what was done for https://github.com/pfsense/pfsense/pull/2765 -
do it to the rest of them.
Seems to work OK.

1 files changed, 2 insertions, 6 deletions

diff --git a/src/usr/local/www/system_groupmanager_addprivs.php b/src/usr/local/www/system_groupmanager_addprivs.php
index d999cdf..5c58092 100644
--- a/src/usr/local/www/system_groupmanager_addprivs.php
+++ b/src/usr/local/www/system_groupmanager_addprivs.php
@@ -217,7 +217,7 @@ $btnfilter = new Form_Button(
 	'fa-filter'
 );
 
-$btnfilter->addClass('btn btn-info');
+$btnfilter->setAttribute('type','button')->addClass('btn btn-info');
 
 $form->addGlobal($btnfilter);
 
@@ -228,7 +228,7 @@ $btnclear = new Form_Button(
 	'fa-times'
 );
 
-$btnclear->addClass('btn btn-warning');
+$btnclear->setAttribute('type','button')->addClass('btn btn-warning');
 
 $form->addGlobal($btnclear);
 $form->add($section);
@@ -287,8 +287,6 @@ events.push(function() {
 		});
 	});
 
-	$('#btnfilter').prop('type', 'button');
-
 	$('#btnfilter').click(function() {
 		searchterm = $('#filtertxt').val().toLowerCase();
 		copyselect(true);
@@ -301,8 +299,6 @@ events.push(function() {
 		});
 	});
 
-	$('#btnclear').prop('type', 'button');
-
 	$('#btnclear').click(function() {
 		// Copy all options from shadow to sysprivs
 		copyselect(true)