getLoadedArticleIds: consider visible rows only

catchupFeedInGroup: hide articles when catching up, with caveats

2 files changed, 21 insertions, 1 deletions

diff --git a/js/feedlist.js b/js/feedlist.js
index a976c55bd..8e1aa6452 100644
--- a/js/feedlist.js
+++ b/js/feedlist.js
@@ -453,8 +453,26 @@ function catchupFeedInGroup(id) {
 
 				rows.each(function (row) {
 					row.removeClassName("Unread");
+
+					if (row.getAttribute("data-article-id") != getActiveArticleId()) {
+						new Effect.Fade(row, {duration: 0.5});
+					}
+
 				});
 
+				var feedTitles = $$("#headlines-frame > div[class='cdmFeedTitle']");
+
+				for (var i = 0; i < feedTitles.length; i++) {
+					if (feedTitles[i].getAttribute("data-feed-id") == id) {
+
+						if (i < feedTitles.length - 1) {
+							new Effect.Fade(feedTitles[i], {duration: 0.5});
+						}
+
+						break;
+					}
+				}
+
 				updateFloatingTitle(true);
 			}
 
diff --git a/js/viewfeed.js b/js/viewfeed.js
index 2873a9e29..8fccd5a49 100755
--- a/js/viewfeed.js
+++ b/js/viewfeed.js
@@ -928,8 +928,10 @@ function getLoadedArticleIds() {
 	var children = $$("#headlines-frame > div[id*=RROW-]");
 
 	children.each(function(child) {
+		if (Element.visible(child)) {
 			rv.push(child.id.replace("RROW-", ""));
-		});
+		}
+	});
 
 	return rv;