emacs icon indicating copy to clipboard operation
emacs copied to clipboard

Support for ediff

Open Linuus opened this issue 5 years ago • 5 comments

It looks like there is no support for ediff yet. Here's a list of faces that zenburn uses: https://github.com/bbatsov/zenburn-emacs/blob/master/zenburn-theme.el#L524-L540

Linuus avatar Feb 17 '20 09:02 Linuus

Hi @Linuus :wave:, thanks for your contribution :+1: I've added it to the backlog, but anyone can feel free to submit a PR for it 😄

arcticicestudio avatar Feb 17 '20 10:02 arcticicestudio

Getting ediff to work with only 16 colours is a big challenge because the default ediff keeps the syntax highlighting and just overlays red etc. What could be done instead is to change the foreground to a constant color (also done in other themes, e.g. solarized) and highlight changes in bold. See attached screenshots. Any opinions regarding this?

nord_ediff_old nord_ediff_new nord_ediff_new2

ssnnoo avatar Aug 26 '20 10:08 ssnnoo

Personally I like the default style more I think. It's a bit easier for me to see what has changed, but maybe I would get used to the other style :)

What about using the default colors but make them a bit lighter/transparent and highlight the inline changes with regular Nord colors (so it more resembles the default ediff but it's a bit more Nord styleish)?

Also, just fixing that hard to read grey highlighting makes a huge difference! :)

Linuus avatar Sep 07 '20 06:09 Linuus

still there are occasions where the default colours do not provide great readability. but overall maybe a good compromise?

newx

ssnnoo avatar Sep 07 '20 10:09 ssnnoo

or like this, but then again we lose the syntax highlighting in the active diff region. personally I don't mind as this is not super useful anyway.

new2

ssnnoo avatar Sep 07 '20 13:09 ssnnoo