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

Inclusion of Base16 themes #35

Open
landroni opened this issue Feb 12, 2014 · 7 comments
Open

Inclusion of Base16 themes #35

landroni opened this issue Feb 12, 2014 · 7 comments

Comments

@landroni
Copy link

Please add support for the Tomorrow themes ( https://github.com/chriskempson/tomorrow-theme#readme ). As far as I go they're pretty excellent and I think that they'd make a great addition to Geany's selection of themes.

@landroni landroni changed the title Provide support for the Tomorrow themes Provide support for the Tomorrow or Base16 themes May 17, 2014
@landroni
Copy link
Author

Actually Base16 is sufficient to cover most of Tomorrow themes. Soon we will make a PR for including base16: https://github.com/RobLoach/base16-geany , but not yet.

@landroni
Copy link
Author

See also: RobLoach/base16-geany#3

@landroni
Copy link
Author

As per RobLoach/base16-geany#4, most issues with the base16-geany themes have been addressed, and now we're ready to make a PR.

Last time we discussed this, you suggested that it would be better for the names to avoid clustering (Base16 3024 Dark) and that instead the names should be of the type: 3024 Dark (Base16). Do you still think that we should rename all themes like this? I'm asking since doing that would create some awkward theme names like Default (Base16). Please let me know how it would be best to proceed.

Also please check the themes at https://github.com/RobLoach/base16-geany/ and try them out to see if any obvious issues emerge. I already did my best to test them as per https://github.com/codebrainz/geany-themes/blob/master/ADDING-A-THEME.md .

@RobLoach
Copy link

Glad we can start the discussion. Didn't even think of including Base16 in Geany itself. Thanks for starting this up, @landroni.

@landroni landroni changed the title Provide support for the Tomorrow or Base16 themes Inclusion of Base16 themes Jun 29, 2014
@yeknava
Copy link

yeknava commented Jul 15, 2014

@landroni
Copy link
Author

I suspect that Base16 already contains Tomorrow Night, even if not by that name. (It's quite possibly the default dark theme.)

@codebrainz
Copy link
Owner

Sorry for the long delay. Are the themes auto-generated? If so (or even if not), is it possible to change the [theme_info] a bit? It would be nice if the names followed geany-themes convention a bit more, I was thinking something like 3024 (dark, base16) or something, so they sort in with the rest of the themes but still people can tell which are the base16 ones. Also the version should be just a number (ex. 0.2.2->22 or some such) because the theme version gets bumped automatically whenever the theme changes. Lastly, there should be a screenshot for each theme so people can see what they look like before downloading, and also for eventually being listed in some time of "catalog" (probably will be put on wiki eventually).

I started doing the above by hand but there's so many themes and it will take a long time to do that manually.

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

No branches or pull requests

4 participants