bbatsov / zenburn-emacs

The Zenburn colour theme ported to Emacs
GNU General Public License v3.0
990 stars 269 forks source link

Modify diff colors #300

Closed xiongtx closed 6 years ago

xiongtx commented 6 years ago

Follow-up on #293.

Note: the diff-changed-* colors have not been touched; I can't figure out a way to make the yellows fit the zenburn-yellow{-,+}*theme. It'd be great if someone wants to take a crack at that.

Highlighted (top, point in region) and unhighlighted (bottom):

image

3-way Ediff:

image

bbatsov commented 6 years ago

I can't figure out a way to make the yellows fit the zenburn-yellow{-,+}*theme

I don't quite get this. The existing 3 colors don't look good or what?

xiongtx commented 6 years ago

The contrast is pretty awful :man_shrugging:.

bbatsov commented 6 years ago

I guess you can use something like color-ligthen/darken to produce darker versions of the yellow (e.g. by steps of something like 10-20%). I'm not expert in colors myself, but this seems reasonable. Or we can use orange?

impaktor commented 6 years ago

Either way, looks to be an improvement. 👍

xiongtx commented 6 years ago

I guess you can use something like color-ligthen/darken to produce darker versions of the yellow

I think I've had enough mucking w/ :rainbow: for now :tired_face:. Let's get this merged and address the yellow later.

impaktor commented 6 years ago

@xiongtx what is it that should be yellow? Is it something derived from vim's zenburn, or the original diff colours? Screenshot?

xiongtx commented 6 years ago

The diff-changed-* faces should be a certain yellow. The default ones work well w/ the diff-added and diff-removed I've implemented here.

Don't have a screenshot handy. You can play around w/ scaling the Zenburn yellows if you like.