Repository navigation
Comment indicator display improvements #11904
Description
Activity
GitMate.io thinks possibly related issues are #10288 (Fix inline avatar display for mentions in comments), #5691 (Datepicker design improvements), #2043 (DI Container improvements), #11522 (Filepicker design improvements), and #11903 (Comment indicator only shown on list entries without avatar (owned by you)).
The comment text could show directly in the tooltip.
That would just work if there was one unread comment, if there would be more of them, this will quickly get to much information for a popover.
Instead of the simple comment icon, we could show the avatar(s) of the people who commented.
We already show avatars of the share owners in the list view, so I think just having the regular comment icon is quite good here. Otherwise it will get confusing which avatar is about sharing and which is about comments.
That would just work if there was one unread comment, if there would be more of them, this will quickly get to much information for a popover.
We could show the latest one, and then below "3 others by
avatars of commenters".We already show avatars of the share owners in the list view, so I think just having the regular comment icon is quite good here. Otherwise it will get confusing which avatar is about sharing and which is about comments.
That's kind of the point: Basically both of these avatars are the same. Some of them just commented (those are likely shown more prominent). The comment icon would then only be an addition to the avatars. Similar to the notifications design adjustment: nextcloud/notifications#183
And for the grid view we already overlap the comments icon (show it instead of avatars) for lack of space.
Seems like this also needs to wait for 17.
6 remaining items
@jancborchardt are you still behind this? Meaning is this still something that should be implemented?
Currently we only show an uninformative tooltip with "1 unread comment" when you hover the icon. You need to click and open the sidebar for more info:
Instead:
@nextcloud/javascript @nextcloud/designers