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

Disable colored usernames for accessibility … #13102

Open
fooness opened this issue Apr 9, 2020 · 7 comments
Open

Disable colored usernames for accessibility … #13102

fooness opened this issue Apr 9, 2020 · 7 comments
Assignees
Labels
T-Enhancement X-Needs-Product More input needed from the Product team

Comments

@fooness
Copy link
Contributor

fooness commented Apr 9, 2020

It would be very good in terms of accessibility to offer some setting option to disable colored usernames.

@turt2live
Copy link
Member

@fooness
Copy link
Contributor Author

fooness commented Apr 9, 2020

@turt2live While I’m okay with color-changing is not an option, I highly advise and encourage a setting which disables colors to increase the legibility and “user experience” for people with color vision deficiency.

Some elements can be styled to suit personal needs, but having absolutely no control over the color of usernames is a little bit problematic.

@fooness
Copy link
Contributor Author

fooness commented Apr 12, 2020

Added an option to manually specify and customize username color in config.json via: matrix-org/matrix-react-sdk#4373

@Fnux
Copy link

Fnux commented Apr 14, 2020

Multiple users at work complained about Riot overusing colors, making the UI rather confusing.

I plan to add (= propose a patchset) a setting to disable to disable username colors in Riot at some point in the next few days. Changing config.json is not a possibility for the common user :-)

@turt2live I believe this issue has not be triaged correctly, can you reopen it?

@turt2live
Copy link
Member

I can reopen it, but as mentioned on the PR this needs product review due to the feature being a security feature.

@turt2live turt2live reopened this Apr 14, 2020
@turt2live turt2live added Z-Community-PR Issue is solved by a community member's PR X-Needs-Product More input needed from the Product team labels Apr 14, 2020
@fooness
Copy link
Contributor Author

fooness commented Apr 14, 2020

@turt2live I fear @Fnux is wishing for some user-side solution to this, like my issue initial asked for. The mentioned PR, which I would really be happy with, might not suit the needs of @Fnux … speaking of myself, I would be very happy with the config.json setting at least until maybe there’s a user-sided solution available at some point …

@bwindels
Copy link
Contributor

bwindels commented May 6, 2020

Note that this is now possible (although not really convenient) by creating a custom theme since matrix-org/matrix-react-sdk#4503 (which didn't make it into 1.6 but will be in the next release), also see https://github.com/vector-im/riot-web/blob/develop/docs/theming.md#custom-themes

@SimonBrandner SimonBrandner removed the Z-Community-PR Issue is solved by a community member's PR label Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Enhancement X-Needs-Product More input needed from the Product team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants