search

tinted-theming / base16-emacs

Base16 themes for Emacs
MIT License
382 stars 76 forks source link

Add more `helm-ff-*` variants #111

Closed ooglyhLL closed 3 years ago

ooglyhLL commented 3 years ago

Support all helm file selection faces and make them closer to both dired and ls colours.

belak commented 3 years ago

Thanks for doing this! Do you have a before/after screenshot? I've been meaning to do helm but I don't personally use it.

ooglyhLL commented 3 years ago

Sure. The original colours didn't match the theme at all, so any support is going to make this better. Primarily it was the face of the highlighted line which confused me. In both screenshots it is the . directory. (screenshot theme is “base16-tomorrow”) [ base16-emacs-before base16-emacs-after

](url)

belak commented 3 years ago

Looks good to me. Thanks!