dbarenholz / halcyon-obsidian

The Halcyon colour scheme by Brittany Chiang, ported to be used in Obsidian.
MIT License
20 stars 2 forks source link

Readability with error messages #1

Open sigrunixia opened 1 year ago

sigrunixia commented 1 year ago

image

The #ffae57 may be a better background for this warning, for readability.

But looking it over, really well done for a release 1. :slight_smile: So far just minor, minor things I see that usually gets added in 2nd and third passes anyways.

Some other areas to look at: Obsidian Sync Icon. Anything toggled with on/off. Typically On/Off should be pretty apparent.

dbarenholz commented 11 months ago

Ah, it appears I never sent my question to this Issue 🙈

Can you elaborate on why "not red" as opposed to "red" is a better colour for "warning" / "off" items?

sigrunixia commented 11 months ago

Checking the colors again. I am more thinking the problem is the text color on that background. It's hard to read. :( It also fails some of the minimum accessibility contrasts for colors.

CleanShot 2023-10-23 at 13 06 05@2x

dbarenholz commented 11 months ago

I do plan to add style settings at some point.

I'm thinking about adding a single switch/slider for "higher contrast" or "contrast mode" that tweaks colors for readability (necessarily meaning they deviate from Halcyon).

That slider setting should, when made, address all readability and accessibility issues.

dbarenholz commented 6 months ago

I didn't forget this! I have worked on adding style settings support in commit ffa401674bea359b18813ac8a1bc4839a1fad60d -- the infrastructure should be there, but I have a lot of testing and modifying descriptions to do before I make the actual release. But: it's coming!