Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

highlight mixed tab/spaces and trailing whitespace #42

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jakimowicz
Copy link

See #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
Copy link
Owner

Sorry for delay in replying.

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

@jakimowicz
Copy link
Author

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

@jasonmp85
Copy link

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
Copy link
Author

On Fri, Apr 26, 2013 at 6:38 AM, Jason Petersen [email protected]:

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//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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants