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

Unlocked-padlock icon is too difficult to see in dark theme #3433

Closed
OliverUv opened this issue Mar 15, 2017 · 6 comments · Fixed by #3479
Closed

Unlocked-padlock icon is too difficult to see in dark theme #3433

OliverUv opened this issue Mar 15, 2017 · 6 comments · Fixed by #3479
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect Z-Platform-Specific

Comments

@OliverUv
Copy link

Description

Unlocked-padlock icon is too difficult to see in dark theme. The one next to the field where the user types their messages.

scrot

Version information

  • Platform: web (in-browser), ,
  • Browser: Firefox 52 64bit
  • OS: Kubuntu Yakkety
  • URL: riot.im/app /
@turt2live
Copy link
Member

#2988

This appears to have been fixed in recent builds (not currently released)

@lampholder
Copy link
Member

This appears to have been fixed for Chrome but not for Firefox; at least on /develop with my firefox it still looks a very similar colour to the background :P

@lampholder lampholder added T-Defect S-Tolerable Low/no impact on users S-Minor Impairs non-critical functionality or suitable workarounds exist P2 Z-Platform-Specific labels Mar 15, 2017
@tessgadwa
Copy link

Relieved that this issue is getting attention. IMO the lime green color used for the locked encryption icon could be used for the unlocked icon as well.

@OliverUv
Copy link
Author

OliverUv commented Mar 15, 2017 via email

@tessgadwa
Copy link

I see your point Oliver. Truly, anything but grey!

@ara4n
Copy link
Member

ara4n commented Mar 25, 2017

fixed by @t3chguy in firefox too - thanks!

@ara4n ara4n closed this as completed Mar 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect Z-Platform-Specific
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants