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

hard to read color in quoted message with sticker reply #3291

Closed
adbenitez opened this issue Jun 29, 2023 · 3 comments · Fixed by #4143
Closed

hard to read color in quoted message with sticker reply #3291

adbenitez opened this issue Jun 29, 2023 · 3 comments · Fixed by #4143
Assignees
Labels
bug Something isn't working good first issue Good for newcomers ui/ux UI/UX related issues
Milestone

Comments

@adbenitez
Copy link
Member

if someone replies a message with a sticker, the author name and vertical bar should have the same color than text, otherwise it is not readable and looks bad

image

I had the impression this issue was solved in the past, maybe it is a regression in the latest version?

@Simon-Laux Simon-Laux added the good first issue Good for newcomers label Jul 11, 2023
@farooqkz farooqkz self-assigned this Jul 12, 2023
@farooqkz farooqkz added bug Something isn't working ui/ux UI/UX related issues labels Jul 19, 2023
@farooqkz
Copy link
Collaborator

depends on #2998

@maxphilippov
Copy link
Collaborator

I pushed #4143 as a possible solution. I tried making it a text color but for me on Windows but the name was blending with text. And I personally find color-coding useful to quickly search through the group dialogs. It might confuse something if that's a message not a quote, so I need feedback on this one.

@Simon-Laux
Copy link
Member

I like the transparency, the current background is consistent with the background for the metadata.
Maybe only change the name to white, but keep the color of the sidebar?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers ui/ux UI/UX related issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants