search

Wowfunhappy / Pebble-RSS-Reader

A full-text RSS Reader for the Pebble Smartwatch
GNU General Public License v3.0
20 stars 1 forks source link

Restore Article → Open Article Selection Menu (long press select) → Go back to article does weird things. #14

Closed Wowfunhappy closed 4 years ago

Wowfunhappy commented 4 years ago

Ugh...

Wowfunhappy commented 4 years ago

Oh, yes, of course!

if (!articleSelectMenuExists && articlePageHistory.length <= 1) {
    if (pageNum > 0) {
        displayArticlePage(articleList, articleNum, pageNum - 1);
    }
    else {
        getArticles(Settings.data('savedFeed'));    
    }
}

Well, once you've pulled up the article selection menu once, articleSelectMenuExists will be true!

I wonder, we may not actually need that check anyway. If we just take out that check and rely entirely on articlePageHistory.length, will everything still work?