From bf3c98384d5de2f319ef19f934fc324eb33f399a Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@bah.spb.su>
Date: Tue, 20 Jan 2009 18:46:21 +0100
Subject: [PATCH] rework subtoolbar actions dropdown

---
 feedlist.js           |  19 ++++++++-
 functions.php         |  91 +++++++++++++++++++++++++-----------------
 images/down_arrow.png | Bin 208 -> 135 bytes
 tt-rss.css            |  36 +++++++++++++++++
 viewfeed.js           |  19 +++++++++
 5 files changed, 128 insertions(+), 37 deletions(-)

diff --git a/feedlist.js b/feedlist.js
index 68c93e4c..177f057c 100644
--- a/feedlist.js
+++ b/feedlist.js
@@ -8,6 +8,7 @@ var mouse_y = 0;
 var mouse_x = 0;
 
 var resize_enabled = false;
+var selection_disabled = false;
 var counters_last_request = 0;
 
 function toggle_sortable_feedlist(enabled) {
@@ -552,6 +553,10 @@ function mouse_move_handler(e) {
 	}
 }
 
+function enable_selection(b) {
+	disable_selection = !b;
+}
+
 function enable_resize(b) {
 	resize_enabled = b;
 }
@@ -569,6 +574,12 @@ function mouse_down_handler(e) {
 			document.onselectstart = function() { return false; };
 			return false;
 		}
+
+		if (selection_disabled) {
+			document.onselectstart = function() { return false; };
+			return false;
+		}
+
 	} catch (e) {
 		exception_error("mouse_move_handler", e);
 	}
@@ -577,7 +588,13 @@ function mouse_down_handler(e) {
 function mouse_up_handler(e) {
 	try {
 		mouse_is_down = false;
-		document.onselectstart = null;
+
+		if (!disable_selection) {
+			document.onselectstart = null;
+			var e = document.getElementById("headlineActionsBody");
+			if (e) Element.hide(e);
+		}
+
 	} catch (e) {
 		exception_error("mouse_move_handler", e);
 	}
diff --git a/functions.php b/functions.php
index 50374dc1..be793c49 100644
--- a/functions.php
+++ b/functions.php
@@ -3978,10 +3978,10 @@
 
 			}
 
-				if (strpos($_SESSION["client.userAgent"], "MSIE") === false) {
+					print "<td class=\"headlineActions$rtl_cpart\">";
 
-					print "<td class=\"headlineActions$rtl_cpart\">
-						<ul class=\"headlineDropdownMenu\">
+
+/*					print"<ul class=\"headlineDropdownMenu\">
 						<li class=\"top2\">
 						".__('Select:')."
 							<a href=\"$sel_all_link\">".__('All')."</a>,
@@ -3998,18 +3998,6 @@
 							<li><span class=\"insensitive\">".__('Mark as read:')."</span></li>
 							<li onclick=\"$catchup_sel_link\">&nbsp;&nbsp;".__('Selection')."</li>";
 
-/*				if (!get_pref($link, 'COMBINED_DISPLAY_MODE')) {
-	
-					print "
-						<li onclick=\"catchupRelativeToArticle(0)\">&nbsp;&nbsp;".__("Above active article")."</li>
-						<li onclick=\"catchupRelativeToArticle(1)\">&nbsp;&nbsp;".__("Below active article")."</li>";
-				} else {
-					print "
-						<li><span class=\"insensitive\">&nbsp;&nbsp;".__("Above active article")."</span></li>
-						<li><span class=\"insensitive\">&nbsp;&nbsp;".__("Below active article")."</span></li>";
-
-				} */
-
 				print "<li onclick=\"$catchup_feed_link\">&nbsp;&nbsp;".__('Entire feed')."</li>";
 
 				//print "<li><span class=\"insensitive\">--------</span></li>";
@@ -4032,29 +4020,60 @@
 					}
 			}
 
-				print	"</ul></li></ul>";
-				print "</td>"; 
-	
-				} else {
-					// old style subtoolbar:
-	
-					print "<td class=\"headlineActions$rtl_cpart\">".
-						__('Select:')."
-									<a href=\"$sel_all_link\">".__('All')."</a>,
-									<a href=\"$sel_unread_link\">".__('Unread')."</a>,
-									<a href=\"$sel_none_link\">".__('None')."</a>
-							&nbsp;&nbsp;".
-							__('Toggle:')." <a href=\"$tog_unread_link\">".__('Unread')."</a>,
-								<a href=\"$tog_marked_link\">".__('Starred')."</a>
-							&nbsp;&nbsp;".
-							__('Mark as read:')."
-								<a href=\"#\" onclick=\"$catchup_page_link\">".__('Page')."</a>,
-								<a href=\"#\" onclick=\"$catchup_feed_link\">".__('Feed')."</a>";
-
-					print "</td>";  
-	
+				print	"</ul></li></ul>"; */
+
+				print __('Select:')."
+							<a href=\"$sel_all_link\">".__('All')."</a>,
+							<a href=\"$sel_unread_link\">".__('Unread')."</a>,
+							<a href=\"$sel_inv_link\">".__('Invert')."</a>,
+							<a href=\"$sel_none_link\">".__('None')."</a></li>";
+
+				print "&nbsp;&nbsp;";
+
+					print "<span 
+						onmouseover=\"enable_selection(false)\" 
+		            onmouseout=\"enable_selection(true)\"
+						onclick=\"toggleHeadlineActions()\" id=\"headlineActionsDrop\">".
+						__("Actions...") . "&nbsp;&nbsp;<img src=\"images/down_arrow.png\">
+						</span>";
+
+				print "<ul id=\"headlineActionsBody\" style=\"display : none\">";
+
+				print "<li><span class=\"insensitive\">".__('Selection toggle:')."</span></li>
+					<li onclick=\"$tog_unread_link\">&nbsp;&nbsp;".__('Unread')."</li>
+					<li onclick=\"$tog_marked_link\">&nbsp;&nbsp;".__('Starred')."</li>
+					<li onclick=\"$tog_published_link\">&nbsp;&nbsp;".__('Published')."</li>
+					<!-- <li><span class=\"insensitive\">--------</span></li> -->
+					<li><span class=\"insensitive\">".__('Mark as read:')."</span></li>
+					<li onclick=\"$catchup_sel_link\">&nbsp;&nbsp;".__('Selection')."</li>";
+
+				print "<li onclick=\"$catchup_feed_link\">&nbsp;&nbsp;".__('Entire feed').
+					"</li>";
+
+				//print "<li><span class=\"insensitive\">--------</span></li>";
+				print "<li><span class=\"insensitive\">".__('Assign label:')."</span></li>";
+
+				$result = db_query($link, "SELECT id, caption FROM ttrss_labels2 WHERE
+					owner_uid = '".$_SESSION["uid"]."' ORDER BY caption");
+
+				while ($line = db_fetch_assoc($result)) {
+
+					$label_id = $line["id"];
+					$label_caption = $line["caption"];
+
+					if ($feed_id < -10 && $feed_id == -11-$label_id) {
+						print "<li onclick=\"javascript:selectionRemoveLabel($label_id)\">
+							&nbsp;&nbsp;$label_caption ".__('(remove)')."</li>";
+					} else {					
+						print "<li onclick=\"javascript:selectionAssignLabel($label_id)\">
+							&nbsp;&nbsp;$label_caption</li>";
+					}
 				}
 
+				print "</ul>";
+
+				print "</td>"; 
+	
 			print "<td class=\"headlineTitle$rtl_cpart\">";
 
 			print "<span id=\"subtoolbar_search\" 
diff --git a/images/down_arrow.png b/images/down_arrow.png
index c2a8c2787d66cbb6d9e74173667879104d3ebb97..ded833282f6d4fc290ec15ae0b1e415157fa6199 100644
GIT binary patch
literal 135
zcmeAS@N?(olHy`uVBq!ia0vp^+(69E!3HGn<!z4zQk(@Ik;Oo9VGw3ym^DWNC}`>F
z;us<!nVg`&XwY=!+<g20_KLQhot-@iM_Wu9M4GM~5Pf`!A-H3qL@e_rrwKA)4n6W4
dvRuSi7%aGj9-fG~y%T5%gQu&X%Q~loCIBL&C0PIf

delta 194
zcmZo?yufJO8Q|y6%O%Cdz`(%k>ERLtr1^oEor4WXUT+mw2U2WF-tI08|3PrU-sO`g
zDoWI|mw5WRvR`Ku<211lXSuEfR3%&D8d2h$pPQSSSHj?2l$uzQnxasiS(2d;l98(5
z?&%xwK1DSTsL;gI#W6(Ua&p1~1_PzyXXoetpFg4P$PpK}HM2M+8`cP}PK#k%kbPvO
m<XwgwDNd%OLk&3$40Yu~5A$?1zXSC%c)GIrxvX<aXaWF1c{|tu

diff --git a/tt-rss.css b/tt-rss.css
index fe8f11a2..48db0f9a 100644
--- a/tt-rss.css
+++ b/tt-rss.css
@@ -2130,3 +2130,39 @@ pre, code {
 	font-size : 12px;
 	font-family : monospace;
 }
+
+span#headlineActionsDrop {
+	border : 1px solid #c0c0c0;
+	padding : 1px 5px 1px 10px;
+	color : #063064;
+	cursor : pointer;
+}
+
+ul#headlineActionsBody {
+	list-style-type : none;
+	position : absolute;
+	margin : 0px;
+	padding : 0px;
+	background-color : white;
+	width : 150px;
+	z-index : 4;
+	top : auto;
+	left : auto;
+	border-width : 1px 1px 1px 1px;
+	border-color : #c0c0c0;
+	border-style : solid;
+	margin-left : -1px;
+	max-height : 250px;
+	overflow : auto;
+}
+
+ul#headlineActionsBody li {
+	padding : 2px 5px 2px 10px;
+	color : black;
+}
+
+ul#headlineActionsBody li:hover {
+	color : #4684ff;
+	cursor : pointer;
+	background : #ecf4ff;
+}
diff --git a/viewfeed.js b/viewfeed.js
index e5f1f8ae..de7f2681 100644
--- a/viewfeed.js
+++ b/viewfeed.js
@@ -2038,3 +2038,22 @@ function show_labels_in_headlines(transport) {
 
 	}
 }
+
+function toggleHeadlineActions() {
+	try {
+		var e = document.getElementById("headlineActionsBody");
+		var p = document.getElementById("headlineActionsDrop");
+
+		if (!Element.visible(e)) {
+			Element.show(e);
+		} else {
+			Element.hide(e);
+		}
+
+		e.style.left = (p.offsetLeft + 1) + "px";
+//		e.style.top = (p.offsetTop + p.offsetHeight - 1) + "px";
+
+	} catch (e) {
+		exception_error("toggleHeadlineActions", e);
+	}
+}
-- 
2.39.5