Article viewport reset after subscription update - ID: 1888784
After the subscription update completes the article viewport automatically
goes to (0,0), which is quite frustrating because one's place is lost.
I don’t know if there are any pre-conditions; I am using the
‘Condensed’ layout, and this seems to occur with all feeds and styles.
Refreshing subscriptions moves current article position - ID: 2032990
When I'm reading an article and Vienna automatically refreshes my
subscriptions, the reader scrolls up to the top as soon as it's done,
losing my place in the article. This is a problem for longer feeds.
are essentially the same bug/feature. This also happens to be a "feature" that bugs me endlessly as it means I can't watch long videos in Vienna without them being reloaded and stopped. So I figured I'd tackle this problem next.
What happens is that when the subscriptions are refreshed the HTML is reloaded into the ArticleView via a call to ArticleView setHTML. Since there may have been changes to an article due to an update of the article with an identical GUID, I'm assuming this was done on purpose. So rather than remove this functionality, I put a simple cache of the last HTML to be placed into the view, if the strings are identical, then setHTML returns before doing anything leaving the existing HTML in place. This fixes both of the above bugs and also doesn't force a reload of videos.
Unless anyone sees a problem with this, I'll send the patch; it's really simple and confined only to ArticleView.h and ArticleView.m.