From e969cc95b2a12a957a8de07a09360a3e3d8e2878 Mon Sep 17 00:00:00 2001
From: ADTC <send2adtc@gmail.com>
Date: Mon, 13 Mar 2017 21:31:02 +0800
Subject: [PATCH] Use icons for Activate/Deactive/Settings links

An unchecked checkbox icon is used for Activate (to show that it's
currently inactive). A checked checkbox icon is used for Deactivate (to
show that it's currently active). A gear icon is used for the Settings
link.

The checkbox icons are placed on the left side of the name, as checkboxes
usually appear. The gear icon is placed floated to the right side of the
name, to have a clean look.

The plugin list is more compact, and it's much easier to identify which
plugins are currently active, and which have settings. No need to read
any words!

The previously used words (with language dependency) is still used as the
tooltip (title) that is shown when the mouse hovers over the icons.

In the themes page, the checked checkbox icon denotes the curently active
theme, but doesn't do anything when clicked. This is by design, since only
one theme can be active at a time. For consistency sake, a circle (radio
button style) was not used.

Signed-off-by: ADTC <send2adtc@gmail.com>
---
 bl-kernel/admin/themes/default/css/default.css |  5 +++++
 bl-kernel/admin/views/plugins.php              | 14 +++++---------
 bl-kernel/admin/views/themes.php               | 10 ++++++----
 3 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/bl-kernel/admin/themes/default/css/default.css b/bl-kernel/admin/themes/default/css/default.css
index 7e5aa9a3..f322c464 100644
--- a/bl-kernel/admin/themes/default/css/default.css
+++ b/bl-kernel/admin/themes/default/css/default.css
@@ -591,6 +591,11 @@ tr.theme-installed {
 	background: #F2F7FF !important;
 }
 
+div.plugin-name i.settings-icon {
+	float: right;
+	margin-top: 3px;
+}
+
 div.plugin-links > a {
 	display: inline-block;
 	margin-top: 5px;
diff --git a/bl-kernel/admin/views/plugins.php b/bl-kernel/admin/views/plugins.php
index bdb13af2..0bd47d67 100644
--- a/bl-kernel/admin/views/plugins.php
+++ b/bl-kernel/admin/views/plugins.php
@@ -20,25 +20,21 @@ foreach($plugins['all'] as $Plugin)
 	echo '
 	<tr '.($Plugin->installed()?'class="plugin-installed"':'class="plugin-notInstalled"').'>
 	<td>
-	<div class="plugin-name">'.$Plugin->name().'</div>
-	<div class="plugin-links">
+	<div class="plugin-name">
 	';
 
 	if($Plugin->installed()) {
+		echo '<a class="uninstall" href="'.HTML_PATH_ADMIN_ROOT.'uninstall-plugin/'.$Plugin->className().'" title="'.$L->g('Deactivate').'"><i class="uk-icon-check-square-o"></i></a> ';
 		if(method_exists($Plugin, 'form')) {
-			echo '<a class="configure" href="'.HTML_PATH_ADMIN_ROOT.'configure-plugin/'.$Plugin->className().'">'.$L->g('Settings').'</a>';
-			echo '<span class="separator"> | </span>';
+			echo '<a class="configure" href="'.HTML_PATH_ADMIN_ROOT.'configure-plugin/'.$Plugin->className().'" title="'.$L->g('Settings').'"><i class="uk-icon-cog settings-icon"></i></a> ';
 		}
-		echo '<a class="uninstall" href="'.HTML_PATH_ADMIN_ROOT.'uninstall-plugin/'.$Plugin->className().'">'.$L->g('Deactivate').'</a>';
 	}
 	else {
-		echo '<a class="install" href="'.HTML_PATH_ADMIN_ROOT.'install-plugin/'.$Plugin->className().'">'.$L->g('Activate').'</a>';
+		echo '<a class="install" href="'.HTML_PATH_ADMIN_ROOT.'install-plugin/'.$Plugin->className().'" title="'.$L->g('Activate').'"><i class="uk-icon-square-o"></i></a> ';
 	}
 
-
-
 	echo '
-	</div>
+	'.$Plugin->name().'</div>
 	</td>';
 
 	echo '<td>';
diff --git a/bl-kernel/admin/views/themes.php b/bl-kernel/admin/views/themes.php
index 56abd3d7..66c7ed39 100644
--- a/bl-kernel/admin/views/themes.php
+++ b/bl-kernel/admin/views/themes.php
@@ -20,16 +20,18 @@ foreach($themes as $theme)
 	echo '
 	<tr '.($theme['dirname']==$Site->theme()?'class="theme-installed"':'class="theme-notInstalled"').'>
 	<td>
-	<div class="plugin-name">'.$theme['name'].'</div>
-	<div class="plugin-links">
+	<div class="plugin-name">
 	';
 
 	if($theme['dirname']!=$Site->theme()) {
-		echo '<a class="install" href="'.HTML_PATH_ADMIN_ROOT.'install-theme/'.$theme['dirname'].'">'.$L->g('Activate').'</a>';
+		echo '<a class="install" href="'.HTML_PATH_ADMIN_ROOT.'install-theme/'.$theme['dirname'].'" title="'.$L->g('Activate').'"><i class="uk-icon-square-o"></i></a> ';
+	}
+	else {
+		echo '<i class="uk-icon-check-square-o"></i> ';
 	}
 
 	echo '
-	</div>
+	'.$theme['name'].'</div>
 	</td>';
 
 	echo '<td>';