From 670090fd3a94f23defde092c9d37c01f4ab2828e Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <machniak@kolabsys.com>
Date: Sun, 26 Feb 2017 19:38:52 +0100
Subject: [PATCH] Fix style for menu items state (active/disabled)

---
 skins/elastic/includes/mail-menu.html | 26 +++++++++++++-------------
 skins/elastic/styles/icons.less       |  3 +++
 skins/elastic/styles/widgets.less     |  6 +++++-
 3 files changed, 21 insertions(+), 14 deletions(-)

diff --git a/skins/elastic/includes/mail-menu.html b/skins/elastic/includes/mail-menu.html
index d2e4844a9..6fa907d9b 100644
--- a/skins/elastic/includes/mail-menu.html
+++ b/skins/elastic/includes/mail-menu.html
@@ -49,8 +49,8 @@
 <div id="forward-menu" class="ui popup">
 	<h3 id="aria-label-forwardmenu" class="voice"><roundcube:label name="arialabelforwardingoptions" /></h3>
 	<ul class="toolbarmenu listing" role="menu" aria-labelledby="aria-label-forwardmenu">
-		<roundcube:button type="link-menuitem" command="forward-inline" label="forwardinline" prop="sub" classAct="forwardlink" class="forwardlink disabled" />
-		<roundcube:button type="link-menuitem" command="forward-attachment" label="forwardattachment" prop="sub" classAct="forwardattachmentlink" class="forwardattachmentlink disabled" />
+		<roundcube:button type="link-menuitem" command="forward-inline" label="forwardinline" prop="sub" classAct="forwardlink" class="forwardlink" />
+		<roundcube:button type="link-menuitem" command="forward-attachment" label="forwardattachment" prop="sub" classAct="forwardattachmentlink active" class="forwardattachmentlink" />
 		<roundcube:container name="forwardmenu" id="forward-menu" />
 	</ul>
 </div>
@@ -67,13 +67,13 @@
 <div id="message-menu" class="ui popup">
 	<h3 id="aria-label-messagemenu" class="voice"><roundcube:label name="arialabelmoremessageactions" /></h3>
 	<ul class="toolbarmenu listing" role="menu" aria-labelledby="aria-label-messagemenu">
-		<roundcube:button type="link-menuitem" command="print" label="printmessage" class="print disabled" classAct="print" />
-		<roundcube:button type="link-menuitem" command="download" label="emlsave" class="download disabled" classAct="download" />
-		<roundcube:button type="link-menuitem" command="edit" prop="new" label="editasnew" class="edit asnew disabled" classAct="edit asnew" />
-		<roundcube:button type="link-menuitem" command="viewsource" label="viewsource" class="source disabled" classAct="source" />
-		<roundcube:button type="link-menuitem" command="move" label="moveto" class="move disabled" classAct="move" innerclass="folder-selector-link" />
-		<roundcube:button type="link-menuitem" command="copy" label="copyto" class="copy disabled" classAct="copy" innerclass="folder-selector-link" />
-		<roundcube:button type="link-menuitem" command="open" label="openinextwin" target="_blank" class="extwin disabled" classAct="extwin" />
+		<roundcube:button type="link-menuitem" command="print" label="printmessage" class="print" classAct="print active" />
+		<roundcube:button type="link-menuitem" command="download" label="emlsave" class="download" classAct="download active" />
+		<roundcube:button type="link-menuitem" command="edit" prop="new" label="editasnew" class="edit asnew" classAct="edit asnew active" />
+		<roundcube:button type="link-menuitem" command="viewsource" label="viewsource" class="source" classAct="source active" />
+		<roundcube:button type="link-menuitem" command="move" label="moveto" class="move" classAct="move active" innerclass="folder-selector-link" />
+		<roundcube:button type="link-menuitem" command="copy" label="copyto" class="copy" classAct="copy active" innerclass="folder-selector-link" />
+		<roundcube:button type="link-menuitem" command="open" label="openinextwin" target="_blank" class="extwin" classAct="extwin active" />
 		<roundcube:container name="messagemenu" id="message-menu" />
 	</ul>
 </div>
@@ -81,10 +81,10 @@
 <div id="markmessage-menu" class="ui popup">
 	<h3 id="aria-label-markmessagemenu" class="voice"><roundcube:label name="arialabelmarkmessagesas" /></h3>
 	<ul class="toolbarmenu listing" role="menu" aria-labelledby="aria-label-markmessagemenu">
-		<roundcube:button type="link-menuitem" command="mark" prop="read" label="markread" classAct="read" class="read disabled" />
-		<roundcube:button type="link-menuitem" command="mark" prop="unread" label="markunread" classAct="unread" class="unread disabled" />
-		<roundcube:button type="link-menuitem" command="mark" prop="flagged" label="markflagged" classAct="flag" class="flag disabled" />
-		<roundcube:button type="link-menuitem" command="mark" prop="unflagged" label="markunflagged" classAct="unflag" class="unflag disabled" />
+		<roundcube:button type="link-menuitem" command="mark" prop="read" label="markread" classAct="read active" class="read" />
+		<roundcube:button type="link-menuitem" command="mark" prop="unread" label="markunread" classAct="unread active" class="unread" />
+		<roundcube:button type="link-menuitem" command="mark" prop="flagged" label="markflagged" classAct="flag active" class="flag" />
+		<roundcube:button type="link-menuitem" command="mark" prop="unflagged" label="markunflagged" classAct="unflag active" class="unflag" />
 		<roundcube:container name="markmenu" id="markmessage-menu" />
 	</ul>
 </div>
diff --git a/skins/elastic/styles/icons.less b/skins/elastic/styles/icons.less
index 0a14f8c87..61c76a946 100644
--- a/skins/elastic/styles/icons.less
+++ b/skins/elastic/styles/icons.less
@@ -208,6 +208,9 @@ button.ui.button.icon:before {
 .toolbarmenu li a.unflag:before {
 	content: @fa-var-flag-o;
 }
+.toolbarmenu li a.filterlink:before {
+	content: @fa-var-filter;
+}
 .toolbar a.button.reply:before {
 	content: @fa-var-mail-reply;
 }
diff --git a/skins/elastic/styles/widgets.less b/skins/elastic/styles/widgets.less
index 844da9f90..d84d7d35b 100644
--- a/skins/elastic/styles/widgets.less
+++ b/skins/elastic/styles/widgets.less
@@ -64,10 +64,14 @@ body.iframe .formbuttons {
 }
 
 .button.disabled,
-.listing li a.disabled {
+.listing li a {
 	opacity: 0.5;
 }
 
+.listing li a.active {
+	opacity: 1;
+}
+
 .listbox .scroller {
 	width: 100%;
 	overflow-x: hidden;