Refined-github: Solution to: review threads are hard to follow

Created on 14 Aug 2020  ·  10Comments  ·  Source: sindresorhus/refined-github

With review threads, often the last comment on the page is not actually the last comment.

How can this be solved?


I'm thinking that we could add a link at the bottom of the conversation pointing to the last review comment. Mockup:

This would be similar to highlight-highest-rated-comment, except it would appear at the very bottom of the page, which is where people expect the latest change.

enhancement under discussion

Most helpful comment

I happen to be 👎 on this change. This would be a "breaking change" I really like the current behavior and I use it all the time to jump to the last comment.

Or just pointing to "Latest comment" bubble which is at the bottom of the conversation would be more acceptable, just like what I have said? In this case, jumping to the “latest changes” just need one more click on the button.

All 10 comments

I come up with another feature request relating to this solution:
Change the update informationupdated <time> ago/ on <date> to updated <time> ago/ on <date> by *** in issue lists if sorted by update date, and make it clickable, linking to "Latest comment" bubble you have mentioned above.
(Would it be a kind of duplicate feature of the open-issue-to-latest-comment🤔? I still hope this feature can be added, though.)

Makes sense, it would still be part of open-issue-to-latest-comment

Actually this feature could be changed to point to the latest visible change on the page, instead of "the bottom-most comment" and would be a partial solution to this issue.

Makes sense, it would still be part of open-issue-to-latest-comment

Actually this feature could be changed to point to the latest visible change on the page, instead of "the bottom-most comment" and would be a partial solution to this issue.

That’s great!

Actually this feature could be changed to point to the latest visible change on the page

I happen to be 👎 on this change. This would be a "breaking change" I really like the current behavior and I use it all the time to jump to the last comment.

Isn't the last comment possibly in a review that's _not_ at the bottom though?

I happen to be 👎 on this change. This would be a "breaking change" I really like the current behavior and I use it all the time to jump to the last comment.

Or just pointing to "Latest comment" bubble which is at the bottom of the conversation would be more acceptable, just like what I have said? In this case, jumping to the “latest changes” just need one more click on the button.

Isn't the last comment possibly in a review that's _not_ at the bottom though?

thats if you are looking at a pull request. for issues many tines I am looking to see a solution and check the last 2 comments is what I am looking for.

The truth is that I still miss https://github.com/sindresorhus/refined-github/pull/1879

I usually end up on a PR review from notifications, which most of the times links directly to a specific comment (which is supposedly the "last edited comment"). In the linear flow, I don't this feature making any difference except for a fancy button that points to the "last edited comment".

IMO PRs are supposed to be non-linear, in terms of comments, but I think that there should be view of activities (updates) on PR in linear order somewhere.

i would prefer something like a timeline style - like the other points in the timelines

just tried to change the exisiting timeline etries from (https://github.com/sindresorhus/refined-github/pull/3479):

Bildschirmfoto 2020-08-21 um 22 57 07

I usually end up on a PR review from notifications, which most of the times links directly to a specific comment (which is supposedly the "last edited comment"). In the linear flow, I don't this feature making any difference except for a fancy button that points to the "last edited comment".

IMO PRs are supposed to be non-linear, in terms of comments, but I think that there should be view of activities (updates) on PR in linear order somewhere.

i also prefer the (email) notifications, to not miss a commit or a comment :(

my suggested timeline view solves also this problem from @notlmn, when there are many new comments in different review threads, then they show up as timeline like in the (email) notifications, but on the timeline

Was this page helpful?
0 / 5 - 0 ratings

Related issues

alexanderadam picture alexanderadam  ·  3Comments

sindresorhus picture sindresorhus  ·  3Comments

Arcanemagus picture Arcanemagus  ·  3Comments

durka picture durka  ·  3Comments

vanniktech picture vanniktech  ·  3Comments