From 0619c9db228ca94339bccd26f109ae2b2b389fa1 Mon Sep 17 00:00:00 2001
From: Stephen Beaver <sbeaver@netgate.com>
Date: Thu, 5 Nov 2015 09:05:52 -0500
Subject: [PATCH] Convert system_certmanager.php to font awesome icons
 Automatically add confirmation dialog to all fa-trash icons

---
 src/usr/local/www/bootstrap/css/pfSense.css |  5 +++++
 src/usr/local/www/firewall_aliases.php      |  2 +-
 src/usr/local/www/jquery/pfSense.js         |  7 ++++---
 src/usr/local/www/system_certmanager.php    | 22 ++++++---------------
 4 files changed, 16 insertions(+), 20 deletions(-)

diff --git a/src/usr/local/www/bootstrap/css/pfSense.css b/src/usr/local/www/bootstrap/css/pfSense.css
index 68d9cde05b..c4e32db145 100644
--- a/src/usr/local/www/bootstrap/css/pfSense.css
+++ b/src/usr/local/www/bootstrap/css/pfSense.css
@@ -327,3 +327,8 @@ form .btn + .btn {
 	vertical-align: middle;
 	margin-right: 6px;
 }
+
+.fa-trash
+{
+	color: red;
+}
\ No newline at end of file
diff --git a/src/usr/local/www/firewall_aliases.php b/src/usr/local/www/firewall_aliases.php
index c90c57354d..272bfac2eb 100644
--- a/src/usr/local/www/firewall_aliases.php
+++ b/src/usr/local/www/firewall_aliases.php
@@ -268,7 +268,7 @@ display_top_tabs($tab_array);
 			</td>
 			<td>
 				<a class="fa fa-pencil" title="<?=gettext("Edit alias"); ?>" href="firewall_aliases_edit.php?id=<?=$i?>"></a>
-				<a class="fa fa-trash"	title="<?=gettext("Delete alias")?>" href="?act=del&amp;tab=<?=$tab?>&amp;id=<?=$i?>" onclick="return confirm('<?=gettext("Are you sure you want to delete this alias?")?>')"></a>
+				<a class="fa fa-trash"	title="<?=gettext("Delete alias")?>" href="?act=del&amp;tab=<?=$tab?>&amp;id=<?=$i?>"></a>
 			</td>
 		</tr>
 <?php endif?>
diff --git a/src/usr/local/www/jquery/pfSense.js b/src/usr/local/www/jquery/pfSense.js
index 1e74d6379f..d346403693 100644
--- a/src/usr/local/www/jquery/pfSense.js
+++ b/src/usr/local/www/jquery/pfSense.js
@@ -119,16 +119,16 @@ $(function() {
 		input.change();
 	});
 	
-	// Add confirm to all btn-danger buttons
+	// Add confirm to all btn-danger buttons and fa-trash icons
 	// Use element title in the confirmation message, or if not available
 	// the element value
-	$('.btn-danger').on('click', function(e){
+	$('.btn-danger, .fa-trash').on('click', function(e){
 		var msg = $.trim(this.textContent);
 		
 		if(!msg)
 			var msg = $.trim(this.value).toLowerCase();
 			
-		var q = 'Are you sure you wish to '+ msg +'?';
+		var q = 'Are you Sure you wish to '+ msg +'?';
 
 		if ($(this).attr('title') != undefined)
 			q = $(this).attr('title')+'?';
@@ -178,6 +178,7 @@ $(function() {
 	$('input[type=checkbox][data-toggle="collapse"]:not(:checked)').each(function() {
 		$( $(this).data('target') ).addClass('collapse');
 	});
+
 	$('input[type=checkbox][data-toggle="disable"]:not(:checked)').each(function() {
 		$( $(this).data('target') ).prop('disabled', true);
 	});
diff --git a/src/usr/local/www/system_certmanager.php b/src/usr/local/www/system_certmanager.php
index 53c8500c8e..0dea2cfe16 100644
--- a/src/usr/local/www/system_certmanager.php
+++ b/src/usr/local/www/system_certmanager.php
@@ -583,7 +583,7 @@ if (!($act == "new" || (($_POST['save'] == gettext("Save")) && $input_errors)))
 			<th><?=gettext("Issuer")?></th>
 			<th><?=gettext("Distinguished Name")?></th>
 			<th><?=gettext("In Use")?></th>
-			<th><?=gettext("Actions")?></th>
+			<th class="col-sm-2"><?=gettext("Actions")?></th>
 		</tr>
 	</thead>
 	<tbody>
@@ -656,24 +656,14 @@ foreach($a_cert as $i => $cert):
 				<?php endif?>
 			</td>
 			<td>
-				<a href="system_certmanager.php?act=exp&amp;id=<?=$i?>" class="btn btn-xs btn-default">
-					<?=gettext("export")?>
-				</a>
-				<a href="system_certmanager.php?act=key&amp;id=<?=$i?>" class="btn btn-xs btn-default">
-					<?=gettext("export key")?>
-				</a>
-				<a href="system_certmanager.php?act=p12&amp;id=<?=$i?>" class="btn btn-xs btn-default">
-					<?=gettext("export p12")?>
-				</a>
+				<a href="system_certmanager.php?act=exp&amp;id=<?=$i?>" class="fa fa-sign-in" title="<?=gettext("Export")?>"></a>
+				<a href="system_certmanager.php?act=key&amp;id=<?=$i?>" class="fa fa-key" title="<?=gettext("Export key")?>"></a>
+				<a href="system_certmanager.php?act=p12&amp;id=<?=$i?>" class="fa fa-key" title="<?=gettext("Export P12")?>"> P12</a>
 				<?php if (!cert_in_use($cert['refid'])): ?>
-					<a href="system_certmanager.php?act=del&amp;id=<?=$i?>" class="btn btn-xs btn-danger">
-						<?=gettext("delete")?>
-					</a>
+					<a href="system_certmanager.php?act=del&amp;id=<?=$i?>" class="fa fa-trash" title="<?=gettext("Delete")?>"></a>
 				<?php endif?>
 				<?php if ($cert['csr']): ?>
-					<a href="system_certmanager.php?act=csr&amp;id=<?=$i?>" class="btn btn-xs btn-default">
-						<?=gettext("update csr")?>
-					</a>
+					<a href="system_certmanager.php?act=csr&amp;id=<?=$i?>" class="fa fa-refresh" title="<?=gettext("Update csr")?>"></a>
 				<?php endif?>
 			</td>
 		</tr>