From 7fdf8eca9900ae626291b9acc65aef2197af1a94 Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Sun, 28 Oct 2012 00:41:51 +0400 Subject: multiple issues fixed in vfeed_group_by_feed --- include/functions.php | 3 +++ js/viewfeed.js | 11 +++++++++-- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/include/functions.php b/include/functions.php index 4c88c0350..656664da2 100644 --- a/include/functions.php +++ b/include/functions.php @@ -2559,9 +2559,12 @@ $offset_query_part = "OFFSET $offset"; } + // proper override_order applied above if ($vfeed_query_part && get_pref($link, 'VFEED_GROUP_BY_FEED', $owner_uid)) { if (!$override_order) { $order_by = "ttrss_feeds.title, $order_by"; + } else { + $order_by = "ttrss_feeds.title, $override_order"; } } diff --git a/js/viewfeed.js b/js/viewfeed.js index 5f7fbee6d..ff09c0a5a 100644 --- a/js/viewfeed.js +++ b/js/viewfeed.js @@ -115,10 +115,16 @@ function headlines_callback2(transport, offset, background, infscroll_req) { c.domNode.removeChild(hsp); $$("#headlines-tmp > div").each(function(row) { - if ($$("#headlines-frame DIV[id="+row.id+"]").length == 0) { + if (row.className == 'cdmFeedTitle') { + row.addClassName('new'); row.style.display = 'none'; c.domNode.appendChild(row); ++num_added; + } else if ($$("#headlines-frame DIV[id="+row.id+"]").length == 0) { + row.style.display = 'none'; + row.addClassName('new'); + c.domNode.appendChild(row); + ++num_added; } else { row.parentNode.removeChild(row); } @@ -145,8 +151,9 @@ function headlines_callback2(transport, offset, background, infscroll_req) { initHeadlinesMenu(); - $$("#headlines-frame > div[id*=RROW]").each( + $$("#headlines-frame > div[class*=new]").each( function(child) { + child.removeClassName('new'); if (!Element.visible(child)) new Effect.Appear(child, { duration : 0.5 }); }); -- cgit v1.2.3