Browse Source

Revert "Fix behavior when marking an article as unread"

This reverts commit 93af0cf61e6e9368888eedb2d2e36397da3f87bd.
Alexis Degrugillier 11 years ago
parent
commit
5617911644
1 changed files with 11 additions and 13 deletions
  1. 11 13
      p/scripts/main.js

+ 11 - 13
p/scripts/main.js

@@ -144,7 +144,6 @@ function mark_read(active, only_not_read) {
 			inc = 0;
 		if (active.hasClass("not_read")) {
 			active.removeClass("not_read");
-			hide_article(active);
 			inc--;
 		} else if (only_not_read !== true || active.hasClass("not_read")) {
 			active.addClass("not_read");
@@ -232,7 +231,14 @@ function toggleContent(new_active, old_active) {
 		}
 		old_active.removeClass("active current");
 		new_active.addClass("current");
-		hide_article(old_active);
+		if (context['auto_remove_article'] && !old_active.hasClass('not_read')) {
+			var p = old_active.prev();
+			var n = old_active.next();
+			if (p.hasClass('day') && n.hasClass('day')) {
+				p.remove();
+			}
+			old_active.remove();
+		}
 	} else {
 		new_active.toggleClass('active');
 	}
@@ -277,17 +283,6 @@ function toggleContent(new_active, old_active) {
 	}
 }
 
-function hide_article(article) {
-	if (context['auto_remove_article'] && !article.hasClass('not_read')) {
-		var p = article.prev();
-		var n = article.next();
-		if (p.hasClass('day') && n.hasClass('day')) {
-			p.remove();
-		}
-		article.remove();
-	}
-}
-
 function prev_entry() {
 	var old_active = $(".flux.current"),
 		new_active = old_active.length === 0 ? $(".flux:last") : old_active.prevAll(".flux:first");
@@ -695,6 +690,9 @@ function init_stream(divStream) {
 	divStream.on('click', '.flux a.read', function () {
 		var active = $(this).parents(".flux");
 		mark_read(active, false);
+		if (context['auto_remove_article']) {
+			active.remove();
+		}
 		return false;
 	});