From bf40c12408a6f6fd3183d54b47ef1fe9e19983d2 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 16:11:44 +0400
Subject: request_counters: force request 50% of the time to keep all counters
 updated

---
 js/feedlist.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/feedlist.js b/js/feedlist.js
index 282e64b37..6f62de7b8 100644
--- a/js/feedlist.js
+++ b/js/feedlist.js
@@ -234,7 +234,7 @@ function request_counters(force) {
 
 			var query = "?op=rpc&method=getAllCounters&seq=" + next_seq();
 
-			if (!force)
+			if (!force && Math.random() >= 0.5)
 				query = query + "&last_article_id=" + getInitParam("last_article_id");
 
 			console.log(query);
-- 
cgit v1.2.3


From 5129d29d65f13323652c236c5990f3983a8ec555 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 16:47:25 +0400
Subject: reduce counters request minimum interval

---
 js/feedlist.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/feedlist.js b/js/feedlist.js
index 6f62de7b8..a9ee19c06 100644
--- a/js/feedlist.js
+++ b/js/feedlist.js
@@ -227,7 +227,7 @@ function request_counters(force) {
 		var date = new Date();
 		var timestamp = Math.round(date.getTime() / 1000);
 
-		if (force || timestamp - counters_last_request > 15) {
+		if (force || timestamp - counters_last_request > 5) {
 			console.log("scheduling request of counters...");
 
 			counters_last_request = timestamp;
-- 
cgit v1.2.3


From 77a3e1233fff8061de3a5c307531d9a4bc7d6dad Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 16:51:06 +0400
Subject: rpc/getAllCounters: include virtual counters when last_article_id is
 supplied and unchanged

---
 classes/rpc.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/classes/rpc.php b/classes/rpc.php
index 46c8b0d85..18df21b16 100644
--- a/classes/rpc.php
+++ b/classes/rpc.php
@@ -247,6 +247,8 @@ class RPC extends Handler_Protected {
 
 		if ($last_article_id != getLastArticleId($this->link)) {
 			$reply['counters'] = getAllCounters($this->link);
+		} else {
+			$reply['counters'] = getVirtCounters($this->link);
 		}
 
 		$reply['runtime-info'] = make_runtime_info($this->link);
-- 
cgit v1.2.3


From 0a7b15207e71acfd219196ff3301f6730f77320f Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 16:57:26 +0400
Subject: remove previous counter hacks; force request counters when there's no
 unread stuff left in current feed (e.g. finished reading)

---
 classes/rpc.php | 2 --
 js/feedlist.js  | 2 +-
 js/viewfeed.js  | 6 ++++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/classes/rpc.php b/classes/rpc.php
index 18df21b16..46c8b0d85 100644
--- a/classes/rpc.php
+++ b/classes/rpc.php
@@ -247,8 +247,6 @@ class RPC extends Handler_Protected {
 
 		if ($last_article_id != getLastArticleId($this->link)) {
 			$reply['counters'] = getAllCounters($this->link);
-		} else {
-			$reply['counters'] = getVirtCounters($this->link);
 		}
 
 		$reply['runtime-info'] = make_runtime_info($this->link);
diff --git a/js/feedlist.js b/js/feedlist.js
index a9ee19c06..b2aadab62 100644
--- a/js/feedlist.js
+++ b/js/feedlist.js
@@ -234,7 +234,7 @@ function request_counters(force) {
 
 			var query = "?op=rpc&method=getAllCounters&seq=" + next_seq();
 
-			if (!force && Math.random() >= 0.5)
+			if (!force)
 				query = query + "&last_article_id=" + getInitParam("last_article_id");
 
 			console.log(query);
diff --git a/js/viewfeed.js b/js/viewfeed.js
index 1cdfdba3f..2f1ba24e0 100644
--- a/js/viewfeed.js
+++ b/js/viewfeed.js
@@ -325,7 +325,8 @@ function article_callback2(transport, id) {
 			Element.show(dijit.byId("net-alert").domNode);
 		}
 
-		request_counters();
+		var unread_in_buffer = $$("#headlines-frame > div[id*=RROW][class*=Unread]").length
+		request_counters(unread_in_buffer == 0);
 
 		headlines_scroll_handler($("headlines-frame"));
 
@@ -1550,7 +1551,8 @@ function cdmClicked(event, id) {
 			openArticleInNewWindow(id);
 		}
 
-		request_counters();
+		var unread_in_buffer = $$("#headlines-frame > div[id*=RROW][class*=Unread]").length
+		request_counters(unread_in_buffer == 0);
 
 	} catch (e) {
 		exception_error("cdmClicked");
-- 
cgit v1.2.3


From cb0810962c30b0adfc818dfdbd53cc5fe3893dcc Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 18:30:33 +0400
Subject: enclosures player: hide html5 audio element

---
 include/functions.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/functions.php b/include/functions.php
index d6c73343e..688bff76c 100644
--- a/include/functions.php
+++ b/include/functions.php
@@ -2796,7 +2796,7 @@
 
 				$id = 'AUDIO-' . uniqid();
 
-				$entry .= "<audio id=\"$id\"\" controls>
+				$entry .= "<audio id=\"$id\"\" controls style='display : none'>
 					<source type=\"$ctype\" src=\"$url\"></source>
 					</audio>";
 
-- 
cgit v1.2.3


From 3af9dbd350163a5761f58802cbca07a6d12f9206 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 18:44:45 +0400
Subject: cdmScrollToArticleId: use more natural scrolling

---
 js/viewfeed.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/js/viewfeed.js b/js/viewfeed.js
index 2f1ba24e0..07af91a67 100644
--- a/js/viewfeed.js
+++ b/js/viewfeed.js
@@ -1132,7 +1132,11 @@ function cdmScrollToArticleId(id) {
 
 		if (!e || !ctr) return;
 
-		ctr.scrollTop = e.offsetTop;
+		if (e.offsetTop+e.offsetHeight > (ctr.scrollTop+ctr.offsetHeight) ||
+				e.offsetTop < ctr.scrollTop) {
+
+			ctr.scrollTop = e.offsetTop;
+		}
 
 	} catch (e) {
 		exception_error("cdmScrollToArticleId", e);
-- 
cgit v1.2.3


From fa9e88c367fb3d6c771aa1788f133087f49696bc Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 22:16:58 +0400
Subject: replace getmicrotime() wrapper with microtime(true)

---
 classes/feeds.php     | 4 ++--
 include/functions.php | 7 +------
 2 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/classes/feeds.php b/classes/feeds.php
index de001d568..886470e90 100644
--- a/classes/feeds.php
+++ b/classes/feeds.php
@@ -152,7 +152,7 @@ class Feeds extends Handler_Protected {
 
 		$reply = array();
 
-		$timing_info = getmicrotime();
+		$timing_info = microtime(true);
 
 		$topmost_article_ids = array();
 
@@ -750,7 +750,7 @@ class Feeds extends Handler_Protected {
 	}
 
 	function view() {
-		$timing_info = getmicrotime();
+		$timing_info = microtime(true);
 
 		$reply = array();
 
diff --git a/include/functions.php b/include/functions.php
index 688bff76c..979a28761 100644
--- a/include/functions.php
+++ b/include/functions.php
@@ -478,11 +478,6 @@
 		print "</select>";
 	}
 
-	function getmicrotime() {
-		list($usec, $sec) = explode(" ",microtime());
-		return ((float)$usec + (float)$sec);
-	}
-
 	function print_radio($id, $default, $true_is, $values, $attributes = "") {
 		foreach ($values as $v) {
 
@@ -3065,7 +3060,7 @@
 	}
 
 	function print_checkpoint($n, $s) {
-		$ts = getmicrotime();
+		$ts = microtime(true);
 		echo sprintf("<!-- CP[$n] %.4f seconds -->", $ts - $s);
 		return $ts;
 	}
-- 
cgit v1.2.3


From 1ebf3b979ecb697285edd3bec284290bd305647d Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 22:20:14 +0400
Subject: replace getmicrotime() wrapper with microtime(true) (2)

---
 backend.php | 2 +-
 public.php  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/backend.php b/backend.php
index 98b76450f..41481a963 100644
--- a/backend.php
+++ b/backend.php
@@ -47,7 +47,7 @@
 
 	startup_gettext();
 
-	$script_started = getmicrotime();
+	$script_started = microtime(true);
 
 	$link = db_connect(DB_HOST, DB_USER, DB_PASS, DB_NAME);
 
diff --git a/public.php b/public.php
index 7138b3d81..dccd77f43 100644
--- a/public.php
+++ b/public.php
@@ -28,7 +28,7 @@
 
 	startup_gettext();
 
-	$script_started = getmicrotime();
+	$script_started = microtime(true);
 
 	$link = db_connect(DB_HOST, DB_USER, DB_PASS, DB_NAME);
 
-- 
cgit v1.2.3


From 104e5c6557707b13307802dd3b9c87d0209d4a92 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 22:27:49 +0400
Subject: moveToPost: implement more natural next/prev moving in combined mode

---
 js/viewfeed.js | 38 ++++++++++++++++++++++++++++++--------
 1 file changed, 30 insertions(+), 8 deletions(-)

diff --git a/js/viewfeed.js b/js/viewfeed.js
index 07af91a67..6f88f7eac 100644
--- a/js/viewfeed.js
+++ b/js/viewfeed.js
@@ -549,13 +549,23 @@ function moveToPost(mode) {
 		}
 
 		if (mode == "next") {
-		 	if (next_id) {
+		 	if (next_id || active_post_id) {
 				if (isCdmMode()) {
 
-					cdmExpandArticle(next_id);
-					cdmScrollToArticleId(next_id);
+					var article = $("RROW-" + active_post_id);
+					var ctr = $("headlines-frame");
 
-				} else {
+					if (article && article.offsetTop + article.offsetHeight >
+							ctr.scrollTop + ctr.offsetHeight) {
+
+						scrollArticle(ctr.offsetHeight/2);
+
+					} else if (next_id) {
+						cdmExpandArticle(next_id);
+						cdmScrollToArticleId(next_id);
+					}
+
+				} else if (next_id) {
 					correctHeadlinesOffset(next_id);
 					view(next_id, getActiveFeedId());
 				}
@@ -563,11 +573,23 @@ function moveToPost(mode) {
 		}
 
 		if (mode == "prev") {
-			if (prev_id) {
+			if (prev_id || active_post_id) {
 				if (isCdmMode()) {
-					cdmExpandArticle(prev_id);
-					cdmScrollToArticleId(prev_id);
-				} else {
+
+					var article = $("RROW-" + active_post_id);
+					var prev_article = $("RROW-" + prev_id);
+					var ctr = $("headlines-frame");
+
+					if (article && article.offsetTop < ctr.scrollTop) {
+						scrollArticle(-ctr.offsetHeight/2);
+					} else if (prev_article && prev_article.offsetTop < ctr.scrollTop) {
+						cdmExpandArticle(prev_id);
+						scrollArticle(-ctr.offsetHeight/2);
+					} else if (prev_id) {
+						cdmExpandArticle(prev_id);
+						cdmScrollToArticleId(prev_id);
+					}
+				} else if (prev_id) {
 					correctHeadlinesOffset(prev_id);
 					view(prev_id, getActiveFeedId());
 				}
-- 
cgit v1.2.3


From c22580b55f959c8deaebf9aa3a7895c7795df75d Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 22:48:43 +0400
Subject: implement classic next/prev article movement behavior, bind to
 ctrl-arrows

---
 include/functions.php | 4 ++++
 js/tt-rss.js          | 6 ++++++
 js/viewfeed.js        | 9 +++++----
 3 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/include/functions.php b/include/functions.php
index 979a28761..8db72f56d 100644
--- a/include/functions.php
+++ b/include/functions.php
@@ -1859,6 +1859,8 @@
 				"prev_feed" => __("Open previous feed"),
 				"next_article" => __("Open next article"),
 				"prev_article" => __("Open previous article"),
+				"next_article_noscroll" => __("Open next article (don't scroll long articles)"),
+				"prev_article_noscroll" => __("Open previous article (don't scroll long articles)"),
 				"search_dialog" => __("Show search dialog")),
 			__("Article") => array(
 				"toggle_mark" => __("Toggle starred"),
@@ -1919,6 +1921,8 @@
 				"p" => "prev_article",
 				"(38)|up" => "prev_article",
 				"(40)|down" => "next_article",
+				"^(38)|Ctrl-up" => "prev_article_noscroll",
+				"^(40)|Ctrl-down" => "next_article_noscroll",
 				"(191)|/" => "search_dialog",
 //			"article" => array(
 				"s" => "toggle_mark",
diff --git a/js/tt-rss.js b/js/tt-rss.js
index 530cb2931..7462d933d 100644
--- a/js/tt-rss.js
+++ b/js/tt-rss.js
@@ -604,6 +604,12 @@ function hotkey_handler(e) {
 		case "prev_article":
 			moveToPost('prev');
 			return false;
+		case "next_article_noscroll":
+			moveToPost('next', true);
+			return false;
+		case "prev_article_noscroll":
+			moveToPost('prev', true);
+			return false;
 		case "search_dialog":
 			search();
 			return ;
diff --git a/js/viewfeed.js b/js/viewfeed.js
index 6f88f7eac..20a3e2b16 100644
--- a/js/viewfeed.js
+++ b/js/viewfeed.js
@@ -514,7 +514,7 @@ function togglePub(id, client_only, no_effects, note) {
 	}
 }
 
-function moveToPost(mode) {
+function moveToPost(mode, noscroll) {
 
 	try {
 
@@ -555,7 +555,7 @@ function moveToPost(mode) {
 					var article = $("RROW-" + active_post_id);
 					var ctr = $("headlines-frame");
 
-					if (article && article.offsetTop + article.offsetHeight >
+					if (!noscroll && article && article.offsetTop + article.offsetHeight >
 							ctr.scrollTop + ctr.offsetHeight) {
 
 						scrollArticle(ctr.offsetHeight/2);
@@ -580,9 +580,10 @@ function moveToPost(mode) {
 					var prev_article = $("RROW-" + prev_id);
 					var ctr = $("headlines-frame");
 
-					if (article && article.offsetTop < ctr.scrollTop) {
+					if (!noscroll && article && article.offsetTop < ctr.scrollTop) {
 						scrollArticle(-ctr.offsetHeight/2);
-					} else if (prev_article && prev_article.offsetTop < ctr.scrollTop) {
+					} else if (!noscroll && prev_article &&
+							prev_article.offsetTop < ctr.scrollTop) {
 						cdmExpandArticle(prev_id);
 						scrollArticle(-ctr.offsetHeight/2);
 					} else if (prev_id) {
-- 
cgit v1.2.3


From c2e4e8fe91c31f76a493033117594c4460025755 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 22:58:43 +0400
Subject: hotkey help: fix actions bound to multiple sequences not displayed
 correctly

---
 classes/backend.php | 35 ++++++++++++++++-------------------
 1 file changed, 16 insertions(+), 19 deletions(-)

diff --git a/classes/backend.php b/classes/backend.php
index 0b6e6bd30..d9a7a9fec 100644
--- a/classes/backend.php
+++ b/classes/backend.php
@@ -23,16 +23,10 @@ class Backend extends Handler {
 		$imap = get_hotkeys_map($this->link);
 		$omap = array();
 
-		// :(
-		$tinycharmap = array(
-			"(9)" => "{TAB}",
-			"(191)" => "?");
-
 		foreach ($imap[1] as $sequence => $action) {
-			if (!isset($omap[$action])) {
-				$omap[$action] = isset($tinycharmap[$sequence]) ? $tinycharmap[$sequence] :
-					$sequence;
-			}
+			if (!isset($omap[$action])) $omap[$action] = array();
+
+			array_push($omap[$action], $sequence);
 		}
 
 		print "<ul class='helpKbList' id='helpKbList'>";
@@ -44,18 +38,21 @@ class Backend extends Handler {
 			print "<li><h3>" . $section . "</h3></li>";
 
 			foreach ($hotkeys as $action => $description) {
-				if (strpos($omap[$action], "|") !== FALSE) {
-					$omap[$action] = substr($omap[$action],
-						strpos($omap[$action], "|")+1,
-						strlen($omap[$action]));
-				}
 
-				print "<li>";
-			 	print "<span class='hksequence'>" . $omap[$action] . "</span>";
-			  	print $description;
-				print "</li>";
-			}
+				foreach ($omap[$action] as $sequence) {
+					if (strpos($sequence, "|") !== FALSE) {
+						$sequence = substr($sequence,
+							strpos($sequence, "|")+1,
+							strlen($sequence));
+					}
+
+					print "<li>";
+				 	print "<span class='hksequence'>$sequence</span>";
+				  	print $description;
+					print "</li>";
 
+				}
+			}
 		}
 
 		print "</ul>";
-- 
cgit v1.2.3


From 67cad82e60f8a2d9b479410babd98cae59c0bc2e Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 27 Feb 2013 23:04:23 +0400
Subject: cdmExpandArticle: do not perform offset manipulation

---
 js/viewfeed.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/js/viewfeed.js b/js/viewfeed.js
index 20a3e2b16..b8dc05aed 100644
--- a/js/viewfeed.js
+++ b/js/viewfeed.js
@@ -1358,12 +1358,12 @@ function cdmExpandArticle(id) {
 			Element.hide("CEXC-" + id);
 		}
 
-		var new_offset = $("RROW-" + id).offsetTop;
+		/* var new_offset = $("RROW-" + id).offsetTop;
 
 		$("headlines-frame").scrollTop += (new_offset-old_offset);
 
 		if ($("RROW-" + id).offsetTop != old_offset)
-			$("headlines-frame").scrollTop = new_offset;
+			$("headlines-frame").scrollTop = new_offset; */
 
 		toggleUnread(id, 0, true);
 		toggleSelected(id);
-- 
cgit v1.2.3


From f4c02a157a51c44c9772e609fcf0e360db42d060 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Thu, 28 Feb 2013 09:13:00 +0400
Subject: fix warning when saving plugins with 0 plugins enabled (closes #542)

---
 classes/pref/prefs.php | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/classes/pref/prefs.php b/classes/pref/prefs.php
index eb84bde99..810b1e164 100644
--- a/classes/pref/prefs.php
+++ b/classes/pref/prefs.php
@@ -824,7 +824,10 @@ class Pref_Prefs extends Handler_Protected {
 	}
 
 	function setplugins() {
-		$plugins = join(",", $_REQUEST["plugins"]);
+		if (is_array($_REQUEST["plugins"]))
+			$plugins = join(",", $_REQUEST["plugins"]);
+		else
+			$plugins = "";
 
 		set_pref($this->link, "_ENABLED_PLUGINS", $plugins);
 	}
-- 
cgit v1.2.3


From 350e3944d49ad26802e27e64057212d88c4d0bf9 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Thu, 28 Feb 2013 09:17:17 +0400
Subject: fix uploadicon() returning wrong content-type (closes #543)

---
 classes/pref/feeds.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/classes/pref/feeds.php b/classes/pref/feeds.php
index da8d68ba1..2ed7b1b24 100644
--- a/classes/pref/feeds.php
+++ b/classes/pref/feeds.php
@@ -437,6 +437,8 @@ class Pref_Feeds extends Handler_Protected {
 	}
 
 	function uploadicon() {
+		header("Content-type: text/html");
+
 		$icon_file = $_FILES['icon_file']['tmp_name'];
 		$feed_id = db_escape_string($_REQUEST["feed_id"]);
 
-- 
cgit v1.2.3