RedReader icon indicating copy to clipboard operation
RedReader copied to clipboard

See Saved Comments

Open ihailfromreddit opened this issue 7 years ago • 2 comments

The Saved Posts view only allows one to see Saved Posts, not Comments. When you go to https://www.reddit.com/user/USERNAME/saved/ on your computer, you can see saved Comments. I would like to be able to see saved comments within RedReader. Thanks.

ihailfromreddit avatar Dec 25 '17 06:12 ihailfromreddit

I looked into this for a little bit, but so far this issue seems very difficult for me to solve. I'll try to explain in as much detail as possible what I found, this will be very technical.

The issue is that the saved posts screen uses a PostListingActivity, which uses PostListingFragment objects to parse all posts(which ignores comments, because it can't parse them). It does not do any parsing by itself. PostListingActivity is used for listing subreddits right now as well as the saved posts screen. These two things make is harder to add support for viewing comments as well in this screen.

My suggestion would be to create a SavedListingActivity class. It could take inspiration from the code of the PostListingActivity, CommentListingActivity and the InboxListingActivity classes. The InboxListingActivity does the parsing by itself without the use of fragments, which makes it able to list both replies and messages. This would seem to be quite complex to me, though.

Adding support for parsing comments to PostListingFragment may also be possible, but that is probably not a desired solution. Another possible solution could also be adding a CommentListingFragment to PostListenActivity, which also seems out of place.

I'll look into this some more, but I can't promise I'll be able to implement a solution.

sharkwouter avatar Oct 05 '19 15:10 sharkwouter

Would parsing and displaying saved posts and saved comments separately like the Infinity For Reddit app make things any easier? It has two tabs in its Saved section, one for posts and one for comments

moboa avatar May 29 '22 18:05 moboa