Request to adjust ivy and swiper match face foreground colours.
bhrgunatha opened this issue · comments
I understand you take the aesthetics of moe very seriously and it's my favourite emacs theme both the dark and the light but I can't live without the dark version!. It's the only theme I've used for several years now.
However using it with swiper and ivy is very difficult to read - the text foreground colour of the matches are far too light to read (in particular the light green and light blue background colours with multiple matches) - they both cause me eye strain. I have to change them in my config but I would like it if you could change them in moe to something with more contrast to make it more readable.
Dark theme example:
Light theme example:
I can easily fork it and change it myself but then I would have to patch it every time it's updated. I know it's not often but it would be better for an upstream fix :)
Hi @bhrgunatha :
Thanks for your using. Are the two screenshots your forked version?
No they were taken with version 20180617.200 from melpa
I've changed the colours in my forked version - commit 7d8618aa to make it more readable but I'm no designer :( and I'm certain you can make better choices than me.
Some suggestions: (top is fg, down is bg) totally by imagination, maybe still need to be modified.
I guess the color order from first matched to last can be: green -> blue -> orange -> magenta
May I take a look of the screenshot of your current version next time?
Of course.
This is an example search of moe-dark-theme.el showing the colour changes I made to it
And here is the same for moe-light-theme.el
moe-light is still not very good contrast though . I suck at colours.
I will try your suggestions tomorrow (more free time then). Thanks.
Here are the result of your suggestions and some changes I made which are much better now:
Dark
Light
What do you think?
I re-forked your repo and added those changes which you can see at commit 6f514e32
About light version:
The background of ivy-current-match should not be same with ivy-minibuffer-match-face-1, maybe setting the latter to yellow-00 would be better.