From e0a7121ba9b0f73b9a9b4a61c4f525fb02331ead Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@bah.spb.su>
Date: Sat, 25 Feb 2006 07:15:05 +0100
Subject: [PATCH] only enable bottom actions toolbar in headlines, when
 headlines list scrolls

---
 backend.php | 16 +++++++++++++---
 viewfeed.js |  9 +++++++++
 2 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/backend.php b/backend.php
index a58cb85e9..3dda8d0bb 100644
--- a/backend.php
+++ b/backend.php
@@ -1385,9 +1385,18 @@
 			return;
 		}
 
-		function print_headline_subtoolbar($link, $feed_site_url, $feed_title) {
+		function print_headline_subtoolbar($link, $feed_site_url, $feed_title, 
+						$bottom = false) {
 
-			print "<table class=\"headlinesSubToolbar\" 
+			if (!$bottom) {
+				$class = "headlinesSubToolbar";
+				$tid = "headlineActionsTop";
+			} else {
+				$class = "invisible";
+				$tid = "headlineActionsBottom";
+			}
+
+			print "<table class=\"$class\" id=\"$tid\"
 				width=\"100%\" cellspacing=\"0\" cellpadding=\"0\"><tr>";
 
 			if (!get_pref($link, 'COMBINED_DISPLAY_MODE')) {
@@ -1580,7 +1589,8 @@
 				print "</table>";
 			}
 
-			print_headline_subtoolbar($link, "javascript:catchupPage()", "Mark page as read");
+			print_headline_subtoolbar($link, 
+				"javascript:catchupPage()", "Mark page as read", true);
 
 
 		} else {
diff --git a/viewfeed.js b/viewfeed.js
index 83a348853..1f0d4d283 100644
--- a/viewfeed.js
+++ b/viewfeed.js
@@ -385,4 +385,13 @@ function init() {
 	if (arguments.callee.done) return;
 	arguments.callee.done = true;		
 	document.onkeydown = hotkey_handler;
+
+	var hw = document.getElementById("headlinesList").scrollHeight;
+	var pw = parent.document.getElementById("headlines").scrollHeight;
+
+	if (hw >= pw) {
+		var bt = document.getElementById("headlineActionsBottom");
+		bt.className = "headlinesSubToolbar";
+	}
+	
 }