Update whitespace-mode colors #13

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

Using latest version from chriskempson/tomorrow-theme. Specifically,
this fixes the issue that tab characters were not being displayed
with tabs in whitespace-style.

@timhillgit timhillgit Update whitespace-mode colors
Using latest version from chriskempson/tomorrow-theme. Specifically,
this fixes the issue that tab characters were not being displayed
with tabs in whitespace-style.
37b09ea
Owner

purcell commented Jan 31, 2014

Hmm, the tabs are being displayed for me without this patch, in cyan.

I guess you mean that when the text is selected, the whitespace indicators aren't visible?

@purcell purcell added a commit that referenced this pull request Jan 31, 2014

@purcell purcell Update whitespace-mode colours
Partly based on work by @timhillgit in #13 - thanks!
506cb12
Owner

purcell commented Jan 31, 2014

Thanks -- these faces really did need some work.

I committed a different set of changes in the end, which I think should at least ensure everything is visible for you. My overall goal is not to exactly match the official tomorrow themes face-for-face: rather, I'm using the same basic colour scheme in a way that makes sense to me.

Cheers!

-Steve

purcell closed this Jan 31, 2014

Hi Steve:

Thanks for your quick response on this. The new faces work great for me. I definitely didn't want to mess with the overall aesthetic; just wanted to see those pesky tabs.

timhillgit deleted the timhillgit:whitespace branch Feb 1, 2014

timhillgit restored the timhillgit:whitespace branch Feb 1, 2014

Owner

purcell commented Feb 1, 2014

Great -- thanks for prodding me to fix this. Happy someone else is using these themes. :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment