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

Change link color for readability? #1713

Closed
Kelrap opened this issue Feb 5, 2025 · 6 comments · Fixed by #1716
Closed

Change link color for readability? #1713

Kelrap opened this issue Feb 5, 2025 · 6 comments · Fixed by #1716

Comments

@Kelrap
Copy link
Collaborator

Kelrap commented Feb 5, 2025

In dark mode, the link color is light, which is difficult to make out against the light message background color.

Image

Changing the color will likely affect its readability in chat previews, so that's something to take into consideration.

Image

Coloring is fine in light mode.

Image
@Kelrap
Copy link
Collaborator Author

Kelrap commented Feb 6, 2025

Link color is now a problem with the dark background of sent-by-other-people messages. Is it possible to keep links from changing color, so it uses whatever color normal text would be in that situation? Or just using different colors in different situations?

Image

@linhtphung
Copy link

Don't see this on web staging yet @ggurdin

Image

@ggurdin
Copy link
Collaborator

ggurdin commented Feb 25, 2025

@Kelrap @linhtphung Are the current colors not visible?

@linhtphung
Copy link

The color of the link is still the same as the text. Not too important, but that's what I saw @ggurdin

@Kelrap
Copy link
Collaborator Author

Kelrap commented Feb 28, 2025

@ggurdin They're visible to the sender, but not the receiver

Image

@ggurdin
Copy link
Collaborator

ggurdin commented Feb 28, 2025

@ggurdin They're visible to the sender, but not the receiver

Image

This will be fixed in the next build

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

Successfully merging a pull request may close this issue.

3 participants