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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adjust info, notice, warning, danger, subtle colors #34258

Draft
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

SierraKomodo
Copy link
Contributor

@SierraKomodo SierraKomodo commented Oct 21, 2023

Changelog

馃啈 SierraKomodo
tweak: Colors used for certain types of in-chat messages have been adjusted. Primarily, there is now a different in color between warning (red -> orange bold italic) and danger (red bold), and notice (blue), info (blue -> purple), and subtle (blue -> gray italic).
/:cl:

Comparison of old to new colors:

Code_x1kB0uEzZT

New colors with color blindness simulations:

ColourSimulations_x64_UccapZp1we

ColourSimulations_x64_zt2HTNx93e

@SierraKomodo SierraKomodo force-pushed the tweak/chat-colors branch 2 times, most recently from 31942ad to 5bd708c Compare October 21, 2023 01:01
@SingingSpock
Copy link
Contributor

Dark mode subtle is barely readable on my display.

@SierraKomodo
Copy link
Contributor Author

SierraKomodo commented Oct 21, 2023

Self todo:

  • Brighten danger text on dark mode
  • Brighten or adjust subtle text on dark mode
  • Re-Apply italics/bold?

@SierraKomodo
Copy link
Contributor Author

Updated with tweaks to colors, and re-added bold and italics stylings.

@SierraKomodo SierraKomodo marked this pull request as ready for review January 9, 2024 17:42
@SierraKomodo
Copy link
Contributor Author

Considering this ready to go. I'll figure out better methods of color blindness accessibility in a separate PR

MuckerMayhem
MuckerMayhem previously approved these changes Jan 10, 2024
@Spookerton
Copy link
Member

I straight up do not like these with the same criticisms as the last time.

@SierraKomodo
Copy link
Contributor Author

Re-drafting to go over Spook's concerns when I get home tonight.

@SierraKomodo SierraKomodo marked this pull request as draft January 11, 2024 15:55
@SierraKomodo SierraKomodo changed the title Adjust info, notice, warning, danger, subtle, success colors Adjust info, notice, warning, danger, subtle colors Jan 22, 2024
@SierraKomodo
Copy link
Contributor Author

Removed the success class, added color blind simulations screencaps.

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

Successfully merging this pull request may close these issues.

None yet

4 participants