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

Use 256 colors if available for base16_tomorrow #115

Closed
wants to merge 1 commit into from

Conversation

tanzeeb
Copy link
Contributor

@tanzeeb tanzeeb commented Oct 14, 2017

The upstream base16-tomorrow-night vim colorscheme uses slightly better
colors if 256 colors are available (indicated by the 'base16colorspace'
variable). This update matches the color selection used in the upstream
colorscheme.

See base16-vim for more details:
https://github.com/chriskempson/base16-vim/blob/master/colors/base16-tomorrow-night.vim

@chrisbra
Copy link
Member

i think those changes should be documented

@chrisbra
Copy link
Member

ping. please amend the commit with the documentation of the base16colorspace variable and squash into a single commit. thanks.

The upstream base16-tomorrow-night vim colorscheme uses slightly better
colors if 256 colors are available (indicated by the 'base16colorspace'
variable). This update matches the color selection used in the upstream
colorscheme.

See base16-vim for more details:
  https://github.com/chriskempson/base16-vim/blob/master/colors/base16-tomorrow-night.vim
@tanzeeb
Copy link
Contributor Author

tanzeeb commented Nov 5, 2017

Thanks @chrisbra, updated the commit message.

@chrisbra
Copy link
Member

please include a small change in the documentation

@chrisbra
Copy link
Member

ping

@chrisbra chrisbra closed this in 4eaa64d Jan 5, 2018
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.

2 participants