Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show recipient avatars only in thread header #4704

Closed
wants to merge 1 commit into from

Conversation

st3iny
Copy link
Member

@st3iny st3iny commented Mar 8, 2021

Ref #4315 (comment)

Requires nextcloud-libraries/nextcloud-vue#1745

A more compact thread participant header.

compact-thread-header

@st3iny
Copy link
Member Author

st3iny commented Mar 8, 2021

Cc @ChristophWurst

@st3iny
Copy link
Member Author

st3iny commented Mar 8, 2021

Is this something we want to have? If so, I'd be happy to move on with the upstream PR :)

@ChristophWurst
Copy link
Member

@nimisha-vijay what do you think? :)

@nimishavijay
Copy link
Member

This looks good actually! Although I am concerned about how informative this would be if a participant had a weird email address, because this does not show the name anywhere. With the way it is right now, we are able to identify each person quickly because their names are very easily visible. What do you think?

@st3iny
Copy link
Member Author

st3iny commented Apr 8, 2021

I agree. It looks nice but losing the ability to quickly grasp participant email addresses isn't worth it.

@st3iny st3iny closed this Apr 8, 2021
@ChristophWurst ChristophWurst deleted the enhancement/noid/compact-recipient-bubble branch April 9, 2021 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants