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

Color contrast for OP and mod names #1346

Open
glyph-se opened this issue May 2, 2024 · 3 comments
Open

Color contrast for OP and mod names #1346

glyph-se opened this issue May 2, 2024 · 3 comments
Assignees
Labels
bug Something isn't working fixed in upcoming release Indicates that an issue has been fixed, and will be released in the next version
Milestone

Comments

@glyph-se
Copy link

glyph-se commented May 2, 2024

Bug Description

Enabling colorized user name or instances doesn't work well together with the background color for OP/mod/self. I think this is an effect of recent changes that applies the colorized name also in the post/comments view.

Screenshot

Expected Behaviour

I expect settings to work together with each other. While I have three proposals as how to resolve this issue, there might be other ways to get it working:

  • Change OP/mod background colors so that they work together with the theme colors
  • Only display the icon next to the name if colors are set, instead of colorizing the whole name.
  • Let OP/mod colors take precedence and override the default color with them

Steps to Reproduce

  1. Change to dark theme
  2. Set user name color to theme primary
  3. Navigate to a post where there comments from OP and/or mods

Additional Context

From this lemmy post.

Similar issue exist with the green background color on your own name, although one might be expected to know that one :)

App Version

0.3.0

Device

Asus Zenfone 9

OS

Android 13

@glyph-se glyph-se added the bug Something isn't working label May 2, 2024
@micahmo
Copy link
Member

micahmo commented May 3, 2024

It looks like this was fixed in 574c285. See this line.

transformColor: (c) => userGroups.isNotEmpty ? theme.textTheme.bodyMedium?.color : c?.withOpacity(opacity),

@CTalvio
Copy link
Collaborator

CTalvio commented May 5, 2024

I got assigned @hjiangsu ?

@hjiangsu
Copy link
Member

Ahh sorry, I assigned it to you because it seems like you fixed the issue as per @micahmo's previous comment!

@hjiangsu hjiangsu added the fixed in upcoming release Indicates that an issue has been fixed, and will be released in the next version label May 11, 2024
@hjiangsu hjiangsu added this to the 0.4.0 milestone May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed in upcoming release Indicates that an issue has been fixed, and will be released in the next version
Projects
None yet
Development

No branches or pull requests

4 participants