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

Avatars in Replies are visually noisy #14794

Closed
polwel opened this issue Jul 28, 2020 · 4 comments
Closed

Avatars in Replies are visually noisy #14794

polwel opened this issue Jul 28, 2020 · 4 comments

Comments

@polwel
Copy link

polwel commented Jul 28, 2020

Description

In replies the name and avatar of the original poster are quoted. They are not easily distinguished from actual messages.

In the screenshot, the entire screen shows messages posted by a single user. The avatar of the OP is further up, off-screen.

The reply disrupts the message, and many readers were confused that the following part of the message was not by the quoted person.

I have redacted the content, in reality the crisp text makes the issue much worse for some reason.

To a lesser extend, the same problem applies to the mention. A mention at the start of the message (as it often is) looks like the mentioned user actually authored the message.

image

While I am at it, I suggest a border/drop-shadow around uploaded images. RN, they blend into the text message a bit too well. Given that there already is a 'Download image' button, they shouldn't.

Suggested changes

Maybe mute the avatar/name in quotes? Make the avatar smaller? In the latter case, the flair icons will probably be too distracting.

Version information

  • Platform: Element Web 1.7.2, hosted on-premises. All browsers.
@polwel
Copy link
Author

polwel commented Jul 28, 2020

Actually, thinking about it some more, it might be best if the quoted message has

  • a smaller avatar,
  • muted user name,
  • muted body text,
  • collapsed body if the length exceeds a few lines.

The horizontal indent of the quote body takes up precious space, and might not be necessary.

@t3chguy t3chguy added the A-Replies reply label Jul 28, 2020
@t3chguy
Copy link
Member

t3chguy commented Jul 28, 2020

Most of your complaints are already tracked: #7633 #7530

@polwel
Copy link
Author

polwel commented Jul 28, 2020

Oh wow, matrix-org/matrix-react-sdk#3553 looks like it will solve all my gripes!

Thanks @t3chguy, I came across the PR when browsing the issues you linked. Fingers crossed it can be merged soon.

@robintown
Copy link
Member

@polwel matrix-org/matrix-react-sdk#3553 has been merged :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants