deplorableword / textmate-solarized

TextMate theme for Solarized
http://ethanschoonover.com/solarized
440 stars 98 forks source link

highlight mixed tab/spaces and trailing whitespace #42

Open jakimowicz opened 11 years ago

jakimowicz commented 11 years ago

See https://github.com/deplorableword/textmate-solarized/issues/17

Using whitespace bundle ( http://mads379.github.com/posts/whitespace-tmbundle ), this patch will highlight trailing whitespaces and mixed tab/spaces.

Selected color may be too vivid, I'm open to suggestions to change it.

deplorableword commented 11 years ago

Sorry for delay in replying.

Could you attach a screenshot of how the new whitespace highlighting looks?

jakimowicz commented 11 years ago

Here it is. Screen Shot 2013-04-02 at 11 29 04 PM

jasonmp85 commented 11 years ago

I'd also really like to see something like this merged.

Is #aa0000 appropriate, though? Shouldn't we use one of the colors from the solarized sets?

jakimowicz commented 11 years ago

On Fri, Apr 26, 2013 at 6:38 AM, Jason Petersen notifications@github.comwrote:

I'd also really like to see something like this merged.

Is #aa0000 appropriate, though? Shouldn't we use one of the colors from the solarized sets?

— Reply to this email directly or view it on GitHubhttps://github.com/deplorableword/textmate-solarized/pull/42#issuecomment-17055341 .

It would probably be a better visual fit. I used this color to be sure I would catch it.

Any suggestion ?

http://fabien.jakimowicz.com