bitcoin-dev-project / fresh-eyes

https://fresheyes.bitcoindevs.xyz
5 stars 3 forks source link

feedback 5/8/24 #37

Closed adamjonas closed 6 months ago

adamjonas commented 6 months ago

The main thing that needs to be updated is the wording of "commented here." Why would I blindly put one of these into my url bar (can't click them)?

Screenshot 2024-05-08 at 2 38 41 PM

Code that is not attached to a line should just be summarized. "There were 23 comments left by 5 reviewers." I think you should probably do the same for the inline comments but I certainly don't want to look at 100 comments not attached to specific lines of code.


Seems like a minor bug that Fresh Eyes combined these two threads into one.

Screenshot 2024-05-08 at 2 45 54 PM Screenshot 2024-05-08 at 2 47 04 PM

I didn't see this elsewhere.