Browse Source

make floating title less cpu intensive

Andrew Dolgov 11 years ago
parent
commit
d1343b844d
1 changed files with 13 additions and 19 deletions
  1. 13 19
      js/viewfeed.js

+ 13 - 19
js/viewfeed.js

@@ -2216,30 +2216,24 @@ function openSelectedAttachment(elem) {
 function updateFloatingTitle() {
 	try {
 		var hf = $("headlines-frame");
+		var child = $("RROW-" + _active_article_id);
 
-		var elems = $$("#headlines-frame > div[id*=RROW]");
+		if (child && child.offsetTop + child.offsetHeight > hf.scrollTop) {
 
-		for (var i = 0; i < elems.length; i++) {
-			var child = elems[i];
+			var header = child.getElementsByClassName("cdmHeader")[0];
 
-			if (child.offsetTop + child.offsetHeight > hf.scrollTop) {
-
-				var header = child.getElementsByClassName("cdmHeader")[0];
-
-				if (child.id != $("floatingTitle").getAttribute("rowid")) {
-					$("floatingTitle").setAttribute("rowid", child.id);
-					$("floatingTitle").innerHTML = header.innerHTML;
-				}
-
-				if (child.offsetTop < hf.scrollTop - header.offsetHeight - 100 &&
-						child.offsetTop + child.offsetHeight - hf.scrollTop > 100)
-					Element.show("floatingTitle");
-				else
-					Element.hide("floatingTitle");
-
-				break;
+			if (child.id != $("floatingTitle").getAttribute("rowid")) {
+				$("floatingTitle").setAttribute("rowid", child.id);
+				$("floatingTitle").innerHTML = header.innerHTML;
 			}
+
+			if (child.offsetTop < hf.scrollTop - header.offsetHeight - 100 &&
+					child.offsetTop + child.offsetHeight - hf.scrollTop > 100)
+				Element.show("floatingTitle");
+			else
+				Element.hide("floatingTitle");
 		}
+
 	} catch (e) {
 		exception_error("updateFloatingTitle", e);
 	}