diff --git a/css/screen.css b/css/screen.css index 15170fc..b635a28 100644 --- a/css/screen.css +++ b/css/screen.css @@ -204,11 +204,11 @@ canvas { .blue #list li { border-bottom: 1px solid #2980b9; } .yellow #list li { border-bottom: 1px solid #f39c12; } -#list li:after { +#list li li:after { content: ""; position: absolute; right: 7px; - top: 0.1em; + top: -0.2em; font-weight: 100; font-size: 3em; font-family: "Entypo"; diff --git a/js/App.js b/js/App.js index ff92dad..6daa9eb 100644 --- a/js/App.js +++ b/js/App.js @@ -22,9 +22,11 @@ App.prototype.authenticate = function() { App.prototype.after_login = function(backend) { - var request = window.navigator.mozApps.getSelf(); - request.onsuccess = function() { - $("#version").innerHTML = request.result.manifest.version; + if (window.navigator.mozApps) { + var request = window.navigator.mozApps.getSelf(); + request.onsuccess = function() { + $("#version").innerHTML = request.result.manifest.version; + } } var _this = this; @@ -37,7 +39,7 @@ App.prototype.after_login = function(backend) { var url = window.location.hash; - if(url == "#list") { + if (url == "#list") { _this.setCurrentRead(); _this.changeToPage("#list"); } else if(url == "#reload") { @@ -127,6 +129,7 @@ App.prototype.after_login = function(backend) { App.prototype.logout = function() { this.backend.logOut(); this.unread_articles = []; + this.next_articles = []; this.populateList(); this.login.log_out(); }; @@ -135,18 +138,26 @@ App.prototype.changeToPage = function(page) { // FIXME var active = $(".active"); - if(active.id == "list") { - this.saveScrollTop = document.body.scrollTop; - } - if(page == "#list") { - document.body.scrollTop = this.saveScrollTop; - } else { - window.scroll(0, 0); + // Save old position + if (active.id == "list") { + this.saveScrollTop = document.documentElement.scrollTop; } + // Switch displays active.removeClass("active"); $(page).addClass("active"); + + if (page == "#list") { + var elem = document.documentElement; + var posit = this.saveScrollTop; + + // Restore old position, after display settles + setTimeout(function() { elem.scrollTop = posit; }, 500); + } else { + // Else top of page + window.scroll(0, 0); + } }; App.prototype.setColor = function(color) { @@ -158,6 +169,7 @@ App.prototype.setColor = function(color) { App.prototype.reload = function() { this.unread_articles = []; + this.next_articles = []; $("#all-read").addClass('inactive'); var number=parseInt(localStorage.numArticles); this.backend.reload(this.gotUnreadFeeds.bind(this),number); @@ -217,17 +229,67 @@ App.prototype.validate = function(articles) { return false; }; +// Utility function to toggle feed content visibility +function toggleFeed(feedid) { + var e = document.getElementById("feed" + feedid.toString()); + if (e.style.display == "none") { + e.style.display = ""; + } else { + e.style.display = "none"; + } +} + App.prototype.populateList = function() { - var html_str = ""; - for (var i = 0; i < this.unread_articles.length; i++) { - var article = this.unread_articles[i]; - html_str += ""; - html_str += ""; - html_str += "

" + article.feed_title + "

"; + // Tabulate all articles, grouped by feed. + // Note that this.unread_articles[] can be growing even + // as our user reads, so we have to leave it alone and + // just point to the appropriate articles in situ. + const ua = this.unread_articles, ual = ua.length; + const byfeed = {}, feeds = []; + for (let x = 0; x < ual; ++x) { + const article = ua[x]; + + // Add this article to an existing feed list, or + // or start one for this feed ID. + const fid = article.feed_id; + if (fid in byfeed) { + byfeed[fid].push(x); + } else { + byfeed[fid] = [x]; + feeds.push( {"fid": fid, "name": article.feed_title} ); + } + } + + // Now build the article list; it's a