-
Notifications
You must be signed in to change notification settings - Fork 347
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
Updated base16 themes #244
Conversation
Uff, that is a lot. Does it really make sense to integrate all of those? Can you please fix those warnings and add the new themes to the documentation please (along with some configuration switches, if those exists, I suppose not). Perhaps, it would also be nice, to add a small section in the help on how those base16 themes have been generated. Thanks! |
I can add the themes to the docs, but regarding how I made the themes, these themes were generated using the templates here: https://github.com/fnurl/base16-vim-airline/blob/master/templates/default.mustache. This is your official repository that base16 links to, and it's used to keep your base16 themes up to date. Unfortunately, it hasn't been used in 4 years; that's why so many themes were added. All of these themes have active maintainers and were approved by Chris Kempson, the official maintainer of Base16. I can link to detailed instructions on generating the themes in the wiki. Regarding the warnings, they're about using single quotes rather than double quotes, but I built the themes from the official theme template. I could modify the theme template locally and rebuild the themes, but the base16 themes would no longer follow the official template, which I feel is even more unclear than using single quotes instead of double quotes. I think the best solution is to get the maintainer of that repository to update the template officially with the styling you want and the configuration switches you mentioned. |
okay, thanks. Then please add a short section in the help and how to update the generated themes. I think then we are ready to include this. |
Hello, would be awesome to see this included! Any way I could help with this? |
@endorama Honestly, I forgot about this PR. I can write the short section in the help without a problem, but I've been procrastinating on creating a formatted list of all the base16 themes for the |
Not sure I understand. That should be pretty easy with some vim magic. First remove all themes starting with base16 from the help page. Then do a |
@ShiromMakkad I can't push to this branch, so I don't know how I could help. Based on your help text I'm willing to provide another PR automating the themes update & the docs, if the maintainer is interested with that. |
....and I was going to make Python script for this. I've got a lot to learn about Vim |
can you please fix the vint warnings? And please squash into a single commit. I think then we are fine. |
Updated base16 themes Updated base16 themes Updated docs with full list of new themes Updated docs with full list of new themes Updated base16 themes Updated base16 themes Updated docs with full list of new themes Updated base16 themes Updated base16 themes Updated docs with full list of new themes
For the warnings,
I had some issues squashing the commits into one, but I think it's good to go. |
what's the reason for messing with the CI scripts? |
Only add the new themes from PR #244 and do not modify the existing themes.
* upstream/master: (65 commits) New themes selenized and selenized_bw Add new theme supernova base16: new themes from PR vim-airline#244 Revert "Updated and re-created base16 themes" Update google_dark.vim Update google_light.vim Move color palette definitions into if-block for old base16_gruvbox_dark_hard gruvbox-dark-hard: allow old style again Updated and re-created base16 themes Add apprentice theme Haddy New Year! Add Google dark and light themes [add] github actions add theme `lessnoise` Update airline-themes.txt Create transparent.vim added power support arch ppc64le on yml file. [add] badwolf's airline term color [add] cool theme's terminal color added nord_minimal.vim ...
Only add the new themes from PR vim-airline#244 and do not modify the existing themes.
We've got 110 new themes!
It looks like the base16 themes haven't been updated in 5 years, and base16 has tripled in size since then. There were also 30 themes that got updated.
Solves #243