emacs/protesilaos.modus-themes
1
0
Fork 0
mirror of https://github.com/protesilaos/modus-themes.git synced 2026-09-10 07:16:30 -04:00
Code Issues Actions Packages Projects Releases Wiki Activity

REVISE current and lazy search colours

Browse source 
The previous combinations were not optimal, as the default green and
cyan are too close to each other.  Depending on factors such
environmental lighting and the quality of the monitor, green and cyan
may be indistinguishable.

The previous combinations was also not ideal when the user option
'modus-themes-deuteranopia' was set to a non-nil value: blue compared to
a subtle teal can also be problematic under certain circumstances.

The new colours (intense yellow for active matches and cool/cyan for
lazy ones) are complementary, meaning that there are naturally easy to
tell apart.  These specific hues are also well-suited for users with
red-green colour deficiency: yellow stays as-is, while the cool colour
becomes a bit more grey though remains distinct.  As such, we do not
need to run the helper function 'modus-themes--deuteran' to set the
style based on the value of 'modus-themes-deuteranopia'.

The new colours do not clash with the style of the 'match' face, nor
with the various permutations of the 'region' face (subject to the user
option 'modus-themes-region').
This commit is contained in:
Protesilaos Stavrou 2022-07-08 15:14:58 +03:00
parent 7ff26d1267
commit c0c542a573
No known key found for this signature in database
GPG key ID: 99BD6459CD5CA3EA
1 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
modus-themes.el
View file

@ -4266,12 +4266,8 @@ by virtue of calling either of `modus-themes-load-operandi' and
((,class ,@(modus-themes--markup magenta-alt magenta-intense bg-alt
bg-special-faint-calm))))
;;;;; search
`(modus-themes-search-success ((,class :inherit ,@(modus-themes--deuteran
'modus-themes-intense-blue
'modus-themes-intense-green))))
`(modus-themes-search-success-lazy ((,class :inherit ,@(modus-themes--deuteran
'modus-themes-special-mild
'modus-themes-refine-cyan))))
`(modus-themes-search-success ((,class :inherit modus-themes-intense-yellow)))
`(modus-themes-search-success-lazy ((,class :inherit modus-themes-special-cold)))
`(modus-themes-search-success-modeline ((,class :foreground ,@(modus-themes--deuteran
blue-active
green-active))))