slide browse feed details

3 files changed, 13 insertions, 9 deletions

diff --git a/modules/pref-feed-browser.php b/modules/pref-feed-browser.php
index 4759a5f6a..b20901279 100644
--- a/modules/pref-feed-browser.php
+++ b/modules/pref-feed-browser.php
@@ -140,7 +140,7 @@
 				$details["title"] ."</a>&nbsp;" .
 				"<span class='subscribers'>($subscribers)</span>";
 			
-			print "<div class=\"browserDetails\" id=\"BRDET-" . $details["id"] . "\">";
+			print "<div class=\"browserDetails\" style=\"display : none\" id=\"BRDET-" . $details["id"] . "\">";
 			print "</div>";
 				
 			print "</li>";
diff --git a/prefs.js b/prefs.js
index 6609bcac2..249a4d55c 100644
--- a/prefs.js
+++ b/prefs.js
@@ -48,8 +48,9 @@ function expand_feed_callback() {
 		try {	
 			var container = document.getElementById("BRDET-" + feed_to_expand);	
 			container.innerHTML=xmlhttp.responseText;
-			container.style.display = "block";
+//			container.style.display = "block";
 //			p_notify("");
+			Effect.SlideDown(container, {duration : 0.5});
 		} catch (e) {
 			exception_error("expand_feed_callback", e);
 		}
@@ -1533,16 +1534,16 @@ function browserToggleExpand(id) {
 
 		var d = document.getElementById("BRDET-" + id);
 
-		if (d.style.display == "block") {		
+/*		if (d.style.display == "block") {		
 			d.style.display = "none";
 			
-		} else {
+		} else { */
 	
+		if (Element.visible(d)) {
+			Effect.SlideUp(d, {duration : 0.5});
+		} else {
 			feed_to_expand = id;
 
-			d.style.display = "block";
-			d.innerHTML = "Loading, please wait...";
-
 			xmlhttp.open("GET", "backend.php?op=pref-feed-browser&subop=details&id="
 				+ param_escape(id), true);
 			xmlhttp.onreadystatechange=expand_feed_callback;
@@ -1550,7 +1551,7 @@ function browserToggleExpand(id) {
 		}
 
 	} catch (e) {
-		exception_error("browserExpand", e);
+		exception_error("browserToggleExpand", e);
 	}
 }
 
diff --git a/tt-rss.css b/tt-rss.css
index 36ae4b1a6..337e3fb9e 100644
--- a/tt-rss.css
+++ b/tt-rss.css
@@ -1035,6 +1035,10 @@ ul.userFeedList {
 	-moz-border-radius : 5px;
 } */
 
+/*#browseBigFeedList li {
+	padding :0px;
+}*/
+
 ul.labelTestResults {
 	height : 100px;
 	overflow : auto;
@@ -1086,7 +1090,6 @@ input.feedBrowseCB {
 
 div.browserDetails {
 	margin : 5px 5px 5px 5px;
-	display : none;
 	padding : 5px;
 }