michael-lazar / rtv

Browse Reddit from your terminal
MIT License
4.64k stars 274 forks source link

Load comments with context #603

Open woorst opened 6 years ago

woorst commented 6 years ago

When opening a comment rather than loading the individual comment, load the the entire comment chain with the comment highlighted. So instead of this you'd have this.

michael-lazar commented 6 years ago

+1, I thought that was the default behavior, I wonder if reddit changed their API.

woorst commented 6 years ago

I'm not sure if it was the default. I know on the website you always had the option of opening the permalink or the link with context.

Changing this line to data['permalink'] = comment._fast_permalink += '?context=99' seems to work well enough. We'd still need to have to relevant comment highlighted/focused or something.

michael-lazar commented 5 years ago

I somewhat misunderstood this issue. At first I thought that this meant that the comment's replies weren't being displayed. Now that I'm looking again, I see that context is for displaying the previous comments that are above the selected comment.

Still a valid issue, but I'm changing it from bug to new feature.