ViennaRSS / vienna-rss

Vienna is a free and open-source RSS/Atom newsreader for macOS.
https://www.vienna-rss.com
Apache License 2.0
1.85k stars 227 forks source link

Unified view issues with Vienna version 3.0.0 Beta 12 and higher #229

Closed cbean closed 10 years ago

cbean commented 11 years ago

I am using:

Layout: unified Filter by: unread articles Style: xcast

With beta 11 I have no problems scrolling windows with tons of articles in any feed. All pictures, text, etc is already there and viewable, immediately.

With versions later than beta 11, i.e. beta 12, 13, 14,... 18 in particular, each rss article is loaded while I am scrolling thru a feed, images are loaded, boxes are resized, and articles are displayed after a long time of rendering. Very often I have to scroll back and forward to make an article render properly. So in my experience unified layout is broken since version 3 beta 12.

I downgraded to version 3 beta 11.

Hopefully, you'll fix this.

Same with Vienna 3 Beta 18.

Echelon9 commented 11 years ago

I believe @barijaona is working on this under his forked branch

barijaona commented 11 years ago

Pull request #233 should improve on this subject.

Note for people interested in helping : we don't use NSTableView/NSTableCellView because NSTableCellView is available only on OS X Lion (10.7) or better. We use PXListView instead.

barijaona commented 10 years ago

Do you see some improvement with Beta 19 ?

cbean commented 10 years ago

@barijaona

I see some speed improvements, but the behavior is still the same while I am scrolling thru a feed, images are loaded, boxes are resized, and articles are displayed after a time of rendering.

The problem is, that not all articles are refreshed in unified list view but only the articles viewable in window. While scrolling thru the articles, they are in the process of refreshing.

Once the feed is deselected and selected again, the articles are rendered / refreshed again. There is no cache...

All in all you can't scroll fast thru articles without seeing refreshing... also there seems a queue for refreshing (viewable in status-bar) and it seems the amount of threads for refreshing articles are limited?

cbean commented 10 years ago

@barijaona

I have a screencast showing the issue (Vienna 3.0.0 Beta 19 Build 40267c3 Issue #229.mov, 30MB), please let me know how to send it to you.

cbean commented 10 years ago

Converted to Vienna 3.0.0 Beta 19 Build 40267c3 Issue #229.mp4, 11.5MB.

barijaona commented 10 years ago

Hi @cbean : I understand your point of view, but this is the result of a deliberate design decision. In former versions of Vienna, Unified view just showed a big and long webpage. This approach had two downsides :

This is why I tested a new approach : instead of generating a single webpage, generate a separate web view for each article. The decision to go ahead with the new approach was collective (pull request #191).

You might have noticed that I continue working on Unified display view. Cocoa requires that anything drawing a view occurs in the main thread. May be we will be able to move certain things in separate threads, but don't expect too much.

cbean commented 10 years ago

@barijaona : Thank your for your answer. I appreciate the new approach and solving issue #162. The web view is nice but not as responsive as it should be.

The downsides of the new web-view:

In my opinion, there should be a caching function so once articles are downloaded, they can be rendered faster and also images are kept and need not to be downloaded. This should also fix re-downloading of contents and images while switching between feeds. Description: FEED1 contains several articles, let's say 100. FEED2 contains several articles, let's say 100. If you select FEED1 all articles are rendered fast, then you select FEED2 (leaving FEED1) all articles of FEED2 are rendered fast. The problem is if you reselect FEED1, all articles are re-rendering and also images aren't kept and re-downloaded. This prevents fast switching between feeds and also fast scrolling in feeds.

So I vote for a caching option or an offline reading option.

Is there a chance to see this in further versions?

cbean commented 10 years ago

Still not working. Still not fluid enough, still reloading stuff which is already viewed. Still bad caching. Still redrawing of articles and bad rendering.

cbean commented 10 years ago

This bug needs to be reopened. Since the issue isn't fix in beta 20. Switched back to Vienna beta 11.