From: Andrew Dolgov Date: Mon, 5 Sep 2005 12:02:00 +0000 (+0100) Subject: iframify main interface, numerous javascript cleanups X-Git-Tag: 1.0.3~45 X-Git-Url: https://git.wh0rd.org/?a=commitdiff_plain;h=f0601b870ceb69b5e29493eac6ad0111e60a8ac4;p=tt-rss.git iframify main interface, numerous javascript cleanups --- diff --git a/backend.php b/backend.php index 89e51716..325c367e 100644 --- a/backend.php +++ b/backend.php @@ -194,13 +194,11 @@ WHERE id = '$id'"); if ($addheader) { - - print " - + print " Tiny Tiny RSS : Article $id - "; + "; } if ($result) { @@ -250,11 +248,22 @@ $skip = $_GET["skip"]; $subop = $_GET["subop"]; $view_mode = $_GET["view"]; + $addheader = $_GET["addheader"]; if (!$skip) $skip = 0; if ($subop == "undefined") $subop = ""; + if ($addheader) { + print " + Tiny Tiny RSS : Article $id + + + + + "; + } + // FIXME: check for null value here $result = pg_query("SELECT *,SUBSTRING(last_updated,1,16) as last_updated, @@ -265,11 +274,11 @@ $line = pg_fetch_assoc($result); - if ($subop == "ForceUpdate" || - (!$subop && $line["update_timeout"] > MIN_UPDATE_TIME)) { - - update_rss_feed($link, $line["feed_url"], $feed); + if ($subop == "ForceUpdate") { +// (!$subop && $line["update_timeout"] > MIN_UPDATE_TIME)) { + update_rss_feed($link, $line["feed_url"], $feed); + } else { if ($subop == "MarkAllRead") { @@ -284,22 +293,26 @@ $feed_last_updated = "Updated: " . $line["last_updated"]; - print " - Search: "; + if (!$addheader) { + + print " + Search: "; - print " Reset"; + print " Reset"; - print "  View: "; + print "  View: "; - print_select("viewbox", $view_mode, array("All Posts", "Starred"), - "onchange=\"javascript:viewfeed('$feed', '0', '');\""); + print_select("viewbox", $view_mode, array("All Posts", "Starred"), + "onchange=\"javascript:viewfeed('$feed', '0', '');\""); - print ""; + print ""; - print " - " . $line["title"] . ""; + print " + " . $line["title"] . ""; + + } $search = $_GET["search"]; @@ -323,6 +336,10 @@ $total_entries = pg_fetch_result($result, 0, "total_entries"); + if (!$addheader) { + $limit_query_part = "LIMIT ".HEADLINES_PER_PAGE." OFFSET $skip"; + } + $result = pg_query("SELECT id,title,updated,unread,feed_id,marked, EXTRACT(EPOCH FROM last_read) AS last_read_ts, @@ -332,7 +349,8 @@ WHERE $search_query_part $view_query_part - feed_id = '$feed' ORDER BY updated DESC LIMIT ".HEADLINES_PER_PAGE." OFFSET $skip"); + feed_id = '$feed' ORDER BY updated DESC + $limit_query_part"); $lnum = 0; @@ -396,45 +414,49 @@ // start unholy navbar block - print ""; - - $next_skip = $skip + HEADLINES_PER_PAGE; - $prev_skip = $skip - HEADLINES_PER_PAGE; - - print "Navigate: "; + if (!$addheader) { - if ($prev_skip >= 0) { + print ""; + + $next_skip = $skip + HEADLINES_PER_PAGE; + $prev_skip = $skip - HEADLINES_PER_PAGE; + + print "Navigate: "; + + if ($prev_skip >= 0) { + print "Previous Page"; + } else { + print "Previous Page"; + } + print " "; + + if ($next_skip < $total_entries) { + print "Next Page"; + } else { + print "Next Page"; + } + print "  Feed: "; + print "Previous Page"; - } else { - print "Previous Page"; - } - print " "; - - if ($next_skip < $total_entries) { + href=\"javascript:viewfeed($feed, 0, 'ForceUpdate');\">Update"; + + print "  Mark as read: "; + + if ($num_unread > 0) { + print "This Page"; + print " "; + } else { + print "This Page"; + print " "; + } + print "Next Page"; - } else { - print "Next Page"; - } - print "  Feed: "; + href=\"javascript:viewfeed($feed, $skip, 'MarkAllRead');\">All Posts"; - print "Update"; - - print "  Mark as read: "; - - if ($num_unread > 0) { - print "This Page"; - print " "; - } else { - print "This Page"; - print " "; } - - print "All Posts"; /* print "  Unmark: "; @@ -464,6 +486,10 @@ print "
$total
"; print "
$unread
"; + if ($addheader) { + print ""; + } + } if ($op == "pref-rpc") { diff --git a/config.php-dist b/config.php-dist index 0860c8f7..dfcc952a 100644 --- a/config.php-dist +++ b/config.php-dist @@ -23,8 +23,5 @@ define(ENABLE_PREFS_CATCHUP_UNCATCHUP, false); // enable "Mark as read/unread" buttons in preferences dialog - define(ENABLE_CONTENT_IFRAME, true); - // show article in an iframe to mitigate scrolling UI issues - ?> diff --git a/functions.js b/functions.js index d7f0ec75..cb241ada 100644 --- a/functions.js +++ b/functions.js @@ -54,6 +54,8 @@ function notify(msg) { var n = document.getElementById("notify"); + if (!n) return; + n.innerHTML = msg; if (msg.length == 0) { @@ -65,8 +67,7 @@ function notify(msg) { } function printLockingError() { - notify("Please wait until operation finishes"); -} + notify("Please wait until operation finishes");} var seq = ""; @@ -102,5 +103,76 @@ function hotkey_handler(e) { } +function cleanSelected(element) { + var content = document.getElementById(element); + + var rows = new Array(); + + for (i = 0; i < content.rows.length; i++) { + content.rows[i].className = content.rows[i].className.replace("Selected", ""); + } + +} + +function getVisibleUnreadHeadlines() { + var content = document.getElementById("headlinesList"); + + var rows = new Array(); + + for (i = 0; i < content.rows.length; i++) { + var row_id = content.rows[i].id.replace("RROW-", ""); + if (row_id.length > 0 && content.rows[i].className.match("Unread")) { + rows.push(row_id); + } + } + return rows; +} + +function getVisibleHeadlineIds() { + + var content = document.getElementById("headlinesList"); + + var rows = new Array(); + + for (i = 0; i < content.rows.length; i++) { + var row_id = content.rows[i].id.replace("RROW-", ""); + if (row_id.length > 0) { + rows.push(row_id); + } + } + return rows; +} + +function getFirstVisibleHeadlineId() { + var rows = getVisibleHeadlineIds(); + return rows[0]; +} + +function getLastVisibleHeadlineId() { + var rows = getVisibleHeadlineIds(); + return rows[rows.length-1]; +} + +function markHeadline(id) { + var row = document.getElementById("RROW-" + id); + if (row) { + row.className = row.className + "Selected"; + } +} + +function getFeedIds() { + var content = document.getElementById("feedsList"); + + var rows = new Array(); + + for (i = 0; i < content.rows.length; i++) { + var id = content.rows[i].id.replace("FEEDR-", ""); + if (id.length > 0) { + rows.push(id); + } + } + + return rows; +} diff --git a/tt-rss.css b/tt-rss.css index 3c3dae03..26d2d98a 100644 --- a/tt-rss.css +++ b/tt-rss.css @@ -42,10 +42,6 @@ table.feedOverview { font-size : small; } -td.headlineToolbar { - padding-top : 10px; -} - table.feeds { font-size : small; } @@ -67,7 +63,17 @@ table.headlinesList td.search { padding-bottom : 3px; */ } -table.headlinesList td.title { +td.headlinesToolbar { + padding : 10px; +} + +td.headlinesTitle { + text-align : right; + font-size : large; + font-weight : bold; +} + +table.headlinesList td.title, table.headlinesHeader td.title { font-weight : bold; font-size : large; border-width : 0px 0px 1px 0px; @@ -135,13 +141,18 @@ table.main td.feeds { padding : 10px; } -table.main td.headlines { - height : 10%; +div.headlineToolbar { + border-width : 1px 0px 0px 0px; + border-color : #c0c0c0; + border-style : solid; padding : 10px; +} + +table.main td.headlines { + height : 25%; border-width : 1px 0px 0px 0px; border-color : #c0c0c0; border-style : solid; - overflow : scroll; } table.main td.prefContent { @@ -152,6 +163,14 @@ table.main td.prefContent { } +table.main td.headlinesToolbarBox { + padding : 0px; + height : 20px; + border-width : 1px 0px 0px 0px; + border-color : #c0c0c0; + border-style : solid; +} + table.main td.content { padding : 0px; border-width : 1px 0px 0px 0px; @@ -375,10 +394,11 @@ div.expPane { margin : 15px; } -iframe.contentFrame { +iframe.contentFrame, iframe.headlinesFrame { width : 100%; border-width : 0px; padding : 0px; margin : 0px; height : 100%; } + diff --git a/tt-rss.js b/tt-rss.js index de262f91..06c24916 100644 --- a/tt-rss.js +++ b/tt-rss.js @@ -16,12 +16,6 @@ var active_offset = false; var total_feed_entries = false; -var _viewfeed_autoselect_first = false; -var _viewfeed_autoselect_last = false; - -var _update_ids; -var _update_num_ids; - var search_query = ""; /*@cc_on @*/ @@ -63,14 +57,7 @@ function feedlist_callback() { } } -function feed_update_callback() { - if (xmlhttp_rpc.readyState == 4) { - var result = xmlhttp_rpc.responseText; - notify(_update_ids); - updateFeed(_update_ids.shift()); - } -} - +/* function viewfeed_callback() { var container = document.getElementById('headlines'); if (xmlhttp.readyState == 4) { @@ -121,15 +108,7 @@ function viewfeed_callback() { notify(""); } -} - -function view_callback() { - var container = document.getElementById('content'); - if (xmlhttp_view.readyState == 4) { - container.innerHTML=xmlhttp_view.responseText; - markHeadline(active_post_id); - } -} +} */ function refetch_callback() { @@ -155,16 +134,6 @@ function updateFeed(feed_id) { } -function scheduleSepFeedUpdate(force) { - notify("Updating feeds in background (M2)..."); - - _update_ids = getFeedIds(); - _update_num_ids = _update_ids.length; - - updateFeed(_update_ids.pop()); - -} - function scheduleFeedUpdate(force) { notify("Updating feeds in background..."); @@ -207,6 +176,7 @@ function updateFeedList(silent, fetch) { } } +/* function catchupPage(feed) { if (!xmlhttp_ready(xmlhttp)) { @@ -266,7 +236,7 @@ function catchupPage(feed) { notify("No unread items on this page."); } -} +} */ function catchupAllFeeds() { @@ -284,6 +254,12 @@ function catchupAllFeeds() { } +function viewCurrentFeed(skip, subop) { + if (active_feed_id) { + viewfeed(active_feed_id, skip, subop); + } +} + function viewfeed(feed, skip, subop) { enableHotkeys(); @@ -306,11 +282,6 @@ function viewfeed(feed, skip, subop) { view_mode = "All Posts"; } -/* if (active_feed_id == feed && subop != "ForceUpdate") { - notify("This feed is currently selected."); - return; - } */ - if (skip < 0 || skip > total_feed_entries) { return; } @@ -327,6 +298,19 @@ function viewfeed(feed, skip, subop) { active_feed_id = feed; active_offset = skip; + if (subop == "MarkAllRead") { + + var feedr = document.getElementById("FEEDR-" + feed); + var feedt = document.getElementById("FEEDT-" + feed); + var feedu = document.getElementById("FEEDU-" + feed); + + feedu.innerHTML = "0"; + + if (feedr.className.match("Unread")) { + feedr.className = feedr.className.replace("Unread", ""); + } + } + var query = "backend.php?op=viewfeed&feed=" + param_escape(feed) + "&skip=" + param_escape(skip) + "&subop=" + param_escape(subop) + "&view=" + param_escape(view_mode); @@ -335,47 +319,25 @@ function viewfeed(feed, skip, subop) { query = query + "&search=" + param_escape(search_query); } - xmlhttp.open("GET", query, true); - xmlhttp.onreadystatechange=viewfeed_callback; - xmlhttp.send(null); - - notify("Loading headlines..."); - -} - -function markHeadline(id) { - var row = document.getElementById("RROW-" + id); - if (row) { - row.className = row.className + "Selected"; - } -} - -function getFeedIds() { - var content = document.getElementById("feedsList"); - - var rows = new Array(); - - for (i = 0; i < content.rows.length; i++) { - var id = content.rows[i].id.replace("FEEDR-", ""); - if (id.length > 0) { - rows.push(id); - } - } + var headlines_frame = document.getElementById("headlines-frame"); + + headlines_frame.src = query + "&addheader=true"; - return rows; -} + var feedr = document.getElementById("FEEDR-" + feed); -function cleanSelected(element) { - var content = document.getElementById(element); + cleanSelected("feedsList"); + feedr.className = feedr.className + "Selected"; - var rows = new Array(); + var ftitle_d = document.getElementById("headlinesTitle"); + var ftitle_s = document.getElementById("FEEDN-" + feed); - for (i = 0; i < content.rows.length; i++) { - content.rows[i].className = content.rows[i].className.replace("Selected", ""); - } + ftitle_d.innerHTML = ftitle_s.innerHTML; + + notify(""); } +/* function view(id,feed_id) { enableHotkeys(); @@ -425,15 +387,11 @@ function view(id,feed_id) { var content = document.getElementById("content-frame"); - if (content) { - content.src = "backend.php?op=view&addheader=true&id=" + param_escape(id); - markHeadline(active_post_id); - } else { - xmlhttp_view.open("GET", "backend.php?op=view&id=" + param_escape(id), true); - xmlhttp_view.onreadystatechange=view_callback; - xmlhttp_view.send(null); - } + content.src = "backend.php?op=view&addheader=true&id=" + param_escape(id); + markHeadline(active_post_id); + } +*/ function timeout() { scheduleFeedUpdate(true); @@ -445,8 +403,12 @@ function resetSearch() { viewfeed(active_feed_id, 0, ""); } -function search(feed) { - viewfeed(feed, 0, ""); +function search() { + if (active_feed_id) { + viewfeed(active_feed_id, 0, ""); + } else { + notify("Please select some feed first."); + } } function localPiggieFunction(enable) { @@ -462,53 +424,7 @@ function localPiggieFunction(enable) { } } -function relativeid_callback() { - - if (xmlhttp_rpc.readyState == 4) { - notify(xmlhttp_rpc.responseText); - } - -} - -function getVisibleUnreadHeadlines() { - var content = document.getElementById("headlinesList"); - - var rows = new Array(); - - for (i = 0; i < content.rows.length; i++) { - var row_id = content.rows[i].id.replace("RROW-", ""); - if (row_id.length > 0 && content.rows[i].className.match("Unread")) { - rows.push(row_id); - } - } - return rows; -} - -function getVisibleHeadlineIds() { - - var content = document.getElementById("headlinesList"); - - var rows = new Array(); - - for (i = 0; i < content.rows.length; i++) { - var row_id = content.rows[i].id.replace("RROW-", ""); - if (row_id.length > 0) { - rows.push(row_id); - } - } - return rows; -} - -function getFirstVisibleHeadlineId() { - var rows = getVisibleHeadlineIds(); - return rows[0]; -} - -function getLastVisibleHeadlineId() { - var rows = getVisibleHeadlineIds(); - return rows[rows.length-1]; -} - +/* function moveToPost(mode) { var rows = getVisibleHeadlineIds(); @@ -547,16 +463,17 @@ function moveToPost(mode) { } } +*/ function localHotkeyHandler(keycode) { - if (keycode == 78) { +/* if (keycode == 78) { return moveToPost('next'); } if (keycode == 80) { return moveToPost('prev'); - } + } */ if (keycode == 82) { return scheduleFeedUpdate(true); @@ -570,37 +487,6 @@ function localHotkeyHandler(keycode) { } -function toggleMark(id, toggle) { - -// notify("Toggle mark: " + id + ", " + toggle); - - if (!xmlhttp_ready(xmlhttp_rpc)) { - printLockingError(); - return; - } - - var mark_img = document.getElementById("FMARKPIC-" + id); - - var query = "backend.php?op=rpc&id=" + id + "&subop=mark"; - - if (toggle == true) { - mark_img.src = "images/mark_set.png"; - mark_img.alt = "Reset mark"; - mark_img.setAttribute('onclick', 'javascript:toggleMark('+id+', false)'); - query = query + "&mark=1"; - } else { - mark_img.src = "images/mark_unset.png"; - mark_img.alt = "Set mark"; - mark_img.setAttribute('onclick', 'javascript:toggleMark('+id+', true)'); - query = query + "&mark=0"; - } - - xmlhttp_rpc.open("GET", query, true); - xmlhttp_rpc.onreadystatechange=rpc_notify_callback; - xmlhttp_rpc.send(null); - -} - function init() { // IE kludge diff --git a/tt-rss.php b/tt-rss.php index 87061338..ec4e8b6e 100644 --- a/tt-rss.php +++ b/tt-rss.php @@ -28,7 +28,7 @@ - +
 
@@ -41,14 +41,41 @@ href="javascript:catchupAllFeeds()">Mark as read

+ + + + +
+ Search: + Reset + +  View: + + + +  Feed: Update + + Mark as read + +
+ + - Please select the feed. + - - - + diff --git a/viewfeed.js b/viewfeed.js new file mode 100644 index 00000000..e8b305dc --- /dev/null +++ b/viewfeed.js @@ -0,0 +1,103 @@ +var active_post_id; +var total_unread = 0; + +var xmlhttp_rpc = false; + +/*@cc_on @*/ +/*@if (@_jscript_version >= 5) +// JScript gives us Conditional compilation, we can cope with old IE versions. +// and security blocked creation of the objects. +try { + xmlhttp = new ActiveXObject("Msxml2.XMLHTTP"); +} catch (e) { + try { + xmlhttp_rpc = new ActiveXObject("Microsoft.XMLHTTP"); + } catch (E) { + xmlhttp_rpc = false; + } +} +@end @*/ + +if (!xmlhttp_rpc && typeof XMLHttpRequest!='undefined') { + xmlhttp_rpc = new XMLHttpRequest(); +} + +function view(id,feed_id) { + + enableHotkeys(); + + var crow = document.getElementById("RROW-" + id); + + if (crow.className.match("Unread")) { + var umark = parent.document.getElementById("FEEDU-" + feed_id); + umark.innerHTML = umark.innerHTML - 1; + crow.className = crow.className.replace("Unread", ""); + + if (umark.innerHTML == "0") { + var feedr = parent.document.getElementById("FEEDR-" + feed_id); + feedr.className = feedr.className.replace("Unread", ""); + } + + total_unread--; + } + + cleanSelected("headlinesList"); + + var upd_img_pic = document.getElementById("FUPDPIC-" + id); + + if (upd_img_pic) { + upd_img_pic.innerHTML = ""; + } + + var unread_rows = getVisibleUnreadHeadlines(); + + if (unread_rows.length == 0) { + var button = document.getElementById("btnCatchupPage"); + if (button) { + button.className = "disabledButton"; + button.href = ""; + } + } + + active_post_id = id; + + var content = parent.document.getElementById("content-frame"); + + if (content) { + content.src = "backend.php?op=view&addheader=true&id=" + param_escape(id); + markHeadline(active_post_id); + } +} + +function toggleMark(id, toggle) { + +// notify("Toggle mark: " + id + ", " + toggle); + + if (!xmlhttp_ready(xmlhttp_rpc)) { + printLockingError(); + return; + } + + var mark_img = document.getElementById("FMARKPIC-" + id); + + var query = "backend.php?op=rpc&id=" + id + "&subop=mark"; + + if (toggle == true) { + mark_img.src = "images/mark_set.png"; + mark_img.alt = "Reset mark"; + mark_img.setAttribute('onclick', 'javascript:toggleMark('+id+', false)'); + query = query + "&mark=1"; + } else { + mark_img.src = "images/mark_unset.png"; + mark_img.alt = "Set mark"; + mark_img.setAttribute('onclick', 'javascript:toggleMark('+id+', true)'); + query = query + "&mark=0"; + } + + xmlhttp_rpc.open("GET", query, true); + xmlhttp_rpc.onreadystatechange=rpc_notify_callback; + xmlhttp_rpc.send(null); + +} + +