Magit highlighting broken

[csdrane]

I think this [1] commit tried to fix the issue but highlighting in magit is still broken for me. The face of the hightlighted text is the same color as the background.

 [1] https://github.com/sellout/emacs-color-theme-solarized/pull/124

[calve]

I assume you are talking of this commit
Are you using the next branch of magit ?

[csdrane]

I'm not clear on how to check my magit version--I believe I'm on the master
trunk. I installed via ELPA. As I read the commit now, the patch was
designed to fix display issues with magit-next. My display issues though
are with master AFAICT.

On Tue, Oct 28, 2014 at 10:47 AM, calve notifications@github.com wrote:

I assume you are talking of this commit
calve@3e0edb9
Are you using the next branch of magit ?

—
Reply to this email directly or view it on GitHub
#126 (comment)
.

[calve]

As far as I know, ELPA do not have the ability to checkout a revision, so you are probably on master since the next branch is yet to be merged.
Maybe try adding (setq solarized-diff-mode "high") to your emacs configuration helps ?

You might try to explore from which faces your magit-diff faces inherits by placing your point where the colors go wrong and M-x describe-face

[csdrane]

It looks like my problem is due to magit-item-highlight, which inherits from secondary-selection, which is defined as black on black for some reason.

Below is an example of the issue I'm experiencing.

[sellout]

I don’t seem to have a magit-item-highlight face on my system, and secondary-selection is reasonably defined (should be base0 on base02).

Could you update to the latest from master and see if you’re still seeing this issue? Thanks.

[sellout]

Sorry – I do have magit-item-highlight and it’s defined as you said (magit just hadn’t been loaded in my session). However, it looks fine to me as does the definition of secondary-selection, so I’m still no help.

[csdrane]

I switched to the Cocoa emacs and everything is gravy now.

On Wed, Jan 7, 2015 at 12:42 AM, Greg Pfeil notifications@github.com
wrote:

Sorry – I do have magit-item-highlight and it’s defined as you said
(magit just hadn’t been loaded in my session). However, it looks fine to me
as does the definition of secondary-selection, so I’m still no help.

—
Reply to this email directly or view it on GitHub
#126 (comment)
.

[sellout]

Great! Thanks for responding.