From 0ceded7a4663fc05d5f4871b01ac66f9c6a7380e Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.spb.ru>
Date: Thu, 18 May 2006 08:56:52 +0100
Subject: [PATCH] interface tweaks

---
 backend.php  |  2 +-
 functions.js | 16 ++++++++--------
 prefs.js     | 12 ++++++------
 tt-rss.css   | 11 +++++++++--
 tt-rss.js    |  2 +-
 5 files changed, 25 insertions(+), 18 deletions(-)

diff --git a/backend.php b/backend.php
index f3c1c58f..be7aba38 100644
--- a/backend.php
+++ b/backend.php
@@ -1797,7 +1797,7 @@
 //				onclick=\"javascript:hideParentElement(this);\" src=\"images/close.png\">";
 	
 			print "<a href=\"javascript:showBlockElement('feedUpdateErrors')\">
-				<b>Feeds with update errors</b> (click to expand)</a>";
+				<b>Some feeds have update errors (click for details)</b></a>";
 
 			print "<ul id=\"feedUpdateErrors\" class=\"nomarks\">";
 						
diff --git a/functions.js b/functions.js
index ae25a2cc..d5ac2c08 100644
--- a/functions.js
+++ b/functions.js
@@ -81,12 +81,12 @@ function p_notify(msg) {
 	if (!n || !nb) return;
 
 	if (msg == "") {
-		nb.innerHTML = "&nbsp;";
-//		n.style.background = "#ffffff";
+		n.style.display = "none";
 	} else {
-		nb.innerHTML = msg;
-//		n.style.background = "#fffff0";
+		n.style.display = "block";
 	}
+	
+	nb.innerHTML = msg;
 }
 
 function notify(msg) {
@@ -97,13 +97,13 @@ function notify(msg) {
 	if (!n || !nb) return;
 
 	if (msg == "") {
-		nb.innerHTML = "&nbsp;";
-//		n.style.background = "#ffffff";
+		n.style.display = "none";
 	} else {
-		nb.innerHTML = msg;
-//		n.style.background = "#fffff0";
+		n.style.display = "block";
 	}
 
+	nb.innerHTML = msg;
+
 }
 
 function printLockingError() {
diff --git a/prefs.js b/prefs.js
index 236c073c..2539c3b6 100644
--- a/prefs.js
+++ b/prefs.js
@@ -281,7 +281,7 @@ function addFeed() {
 	var link = document.getElementById("fadd_link");
 
 	if (link.value.length == 0) {
-		alert("Can't add feed: missing feed URL.");
+		alert("Error: No feed URL given.");
 	} else {
 		notify("Adding feed...");
 
@@ -643,7 +643,7 @@ function feedEditCancel() {
 
 	active_feed = false;
 
-	notify("Operation cancelled.");
+//	notify("Operation cancelled.");
 
 /*	xmlhttp.open("GET", "backend.php?op=pref-feeds", true);
 	xmlhttp.onreadystatechange=feedlist_callback;
@@ -660,7 +660,7 @@ function feedCatEditCancel() {
 
 	active_feed_cat = false;
 
-	notify("Operation cancelled.");
+//	notify("Operation cancelled.");
 
 	xmlhttp.open("GET", "backend.php?op=pref-feeds", true);
 	xmlhttp.onreadystatechange=feedlist_callback;
@@ -799,7 +799,7 @@ function labelEditCancel() {
 
 	active_label = false;
 
-	notify("Operation cancelled.");
+//	notify("Operation cancelled.");
 
 	xmlhttp.open("GET", "backend.php?op=pref-labels", true);
 	xmlhttp.onreadystatechange=labellist_callback;
@@ -816,7 +816,7 @@ function userEditCancel() {
 
 	active_user = false;
 
-	notify("Operation cancelled.");
+//	notify("Operation cancelled.");
 
 	xmlhttp.open("GET", "backend.php?op=pref-users", true);
 	xmlhttp.onreadystatechange=userlist_callback;
@@ -833,7 +833,7 @@ function filterEditCancel() {
 
 	active_filter = false;
 
-	notify("Operation cancelled.");
+//	notify("Operation cancelled.");
 
 	xmlhttp.open("GET", "backend.php?op=pref-filters", true);
 	xmlhttp.onreadystatechange=filterlist_callback;
diff --git a/tt-rss.css b/tt-rss.css
index 9d38cf4a..b1b2a23a 100644
--- a/tt-rss.css
+++ b/tt-rss.css
@@ -296,9 +296,16 @@ td.notifyBox {
 }
 
 #notify {
-	color : black;
-	font-weight : bold;
+	background : #fff7d5;
+	border : 1px solid #d7c47a;
 	padding : 3px;
+	top : 2px;
+	right : 2px;
+	float : right;	
+	position : absolute;	
+	-moz-border-radius : 3px;
+	font-size : x-small;
+	font-weight : bold;
 }
 
 #dispSwitch {
diff --git a/tt-rss.js b/tt-rss.js
index 31ad8a96..f3f496b1 100644
--- a/tt-rss.js
+++ b/tt-rss.js
@@ -534,7 +534,7 @@ function qafAdd() {
 	var link = document.getElementById("qafInput");
 
 	if (link.value.length == 0) {
-		notify("Missing feed URL.");
+		alert("Error: No feed URL given.");
 	} else {
 		notify("Adding feed...");
 	
-- 
2.39.2