vscode-pull-request-github icon indicating copy to clipboard operation
vscode-pull-request-github copied to clipboard

GitHub Notifications UX

Open benibenj opened this issue 3 years ago • 3 comments

For the GitHub notifications we need an indication of a notification belonging to a PR in addition to the highlighted color.

Here are two examples for right aligned notification badges:

notifications1 notifications2

Another option is having the icon in the front (We currently also have the checkmark in the front to indicate the checked out PR):

notifications3

benibenj avatar Aug 03 '22 16:08 benibenj

Talked about this at the Redmond standup:

This version received positive feedback but there was some concern that it would look too similar to the git decorations. CleanShot 2022-08-04 at 16 32 56

This one maybe feels a bit noisier inlined but it could help clarify that it isn't related to git: CleanShot 2022-08-04 at 16 33 09

Also wanted to understand if I'm interpreting the idea correctly—would we show a dot on both the PR title and the actual file with the comment? And then both would clear when the last file with a unread notification was clicked?

FWIW this is using the circle-filled icon.

daviddossett avatar Aug 04 '22 23:08 daviddossett

Thanks for the feedback @daviddossett. The notification will only be indicated on the PR node and not on any files. Due to this I don't think it will be a big problem on the right, as there is no git indication on that node.

I believe we can't use the circle-filled icon in this case as it's a badge which only takes a string as input.

benibenj avatar Aug 05 '22 09:08 benibenj

Oh ok, that simplifies things. Let's give that a try then!

daviddossett avatar Aug 05 '22 16:08 daviddossett

Close with #3839

benibenj avatar Aug 12 '22 08:08 benibenj