-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Task: Create theme maintainers team and see who is still wanting to help maintain themes #3118
Comments
@ericwbailey Are you still interested in ownership and maintenance of the |
@taufik-nurrohman Still interested in ownership and maintenance of agate? |
@gusbemacbe Still interested in ownership and maintenance of an-old-hope? |
@kanytu Still interested in ownership/maintenance of the androidstudio theme? |
@smellai Still interested in ownership/maintenance of the arduino-light theme? |
@atelierbram Still interested in ownership/maintenance of all the atelier-* themes that seem to be your namesake? :) |
Can I see the repo? |
All the styles are in the same folder: https://github.com/highlightjs/highlight.js/tree/main/src/styles |
Ah, I saw. Yes, @joshgoebel . |
Yes, Josh, I'm. |
@voldmar Still interested in ownership and maintenance of zenburn theme? |
Sure 👍 |
@initbar Still interested in ownership and maintenance of xt256.css? |
@joshgoebel sure I am ... |
@joshgoebel Yup. It'd be helpful to have a punchlist for changes once they happen, as I don't actively monitor this repo. |
I'll no longer maintain this theme. |
I guess I won’t maintain it. Sorry |
Yeah, once I have a team together I can CC the team directly on things that might be relevant, etc... or ask for feedback. Right now there is no way to even notify someone that their theme might need an update, etc. |
@angelolloqui Still wanting to own/maintain the Xcode theme? |
@Nicolas01 Are you still wishing to own/maintain the VS Code 2015 dark theme? |
@jdiamond Still wishing to own/maintain the vs.css theme? |
@redguardtoo Still wishing to own/maintain the Srcery theme? |
@thingsinjars Still wishing to own/maintain docco.css theme? |
@DiVAN1x Still wishing to own and maintain the routers.css theme? |
@gavsiu Still wishing to own and maintain the Ocean Dark theme? |
@RocketMan Still wishing to own/maintain nnfx/nnfxdark? |
@ourmaninamsterdam Might you still be interesting to own and maintain the Codepen Embed theme moving forward? I see you had the original commit. |
@Hirse Might you still be interesting to own and maintain the Stack Overflow themes moving forward? I see you had the original commit. Also if you're interested in owning any of the other themes without a home, let me know. :) Should be minimal effort over time. |
Yep. I'll continue supporting it. No problem. |
Sure. |
@atelierbram If we just pull in all your themes via https://github.com/atelierbram/base16-atelier-schemes instead of maintaining by hand does that solve that? |
Sure |
Hi @joshgoebel , I am sorry but I don't have time to maintain the VS Code 2015 dark theme for the moment. |
Sure. I can also take over the GitHub theme unless you have someone for it already. |
@Hirse Do we still need GitHub and Github Gist? I looked and after the recent changes they look almost identical to me? |
Yes, I would be interested |
CC @highlightjs/theme-maintainers #3139 pulls in ALL official Base16 themes via the automated build from https://github.com/highlightjs/base16-highlightjs. Any manual ports of Base16 themes (listed at the top of this thread) have been removed in favor of the new canonical auto-builds. For some themes this may result in visual changes. Just to name one I took a look at Gruvbox Dark and while it was using the same Base16 color palette as the canonical Gruvbox dark - they were all over the map in their application (ie, not applied according to the Base16 spec as far as I could tell). If this has truly detrimental effects on any themes (that perhaps were perhaps higher-fidelity before) this can be reviewed on a case-by-case basis to decide what to do. After the current slate of PRs are reviewed a Or if you want to see just the new themes changes now see #3139. |
@joshgoebel I guess it will be OK |
@joshgoebel I'm the author of Hopscotch and both Kimbie and Paraíso themes. I'm open to maintaining them in the future. |
@idleberg Awesome.
Just out of curiosity is there a reason why you haven't released Kimbie and Paraiso as Base16 themes? |
While those themes were built using |
Thanks for the background.
I think you mean "schemes"? That's what Base16 calls themes... "template" means the actual "destination"... such as now we actually have a Highlight.js template - which turns Base16 schemes into something workable for us. [edit: but perhaps we had one long ago just it wasn't saved anywhere - or there are actually multiple templates and I just don't know where they are, lol]
Ah, I hadn't considered the history, that Highlight.js might predate Base16?
Unless someone steps up to go thru every theme one by one (which doesn't seem likely)... I'm going to assume the Base16 "spec" is canonical. So going forward the Base16 themes we bundle will all use our official template and will adhere to the Base16 guidelines. In the cases where this vastly alters the appearance of themes that did not follow the guidelines before the prior behavior will be considered a bug - and the new behavior the fix. In cases where a specific theme maintainer steps forward because their theme has been mangled by this decision then we can deal with that on a one off basis. It's possible that (now original) theme could then be restored or renamed or something... not sure. The only original Base16 theme I've left (for now) is Nord which (for right or wrong) contains a zillion per-grammar style alterations. So right now you can find both
Oh wow this just registered with me... damn it'd be nice to see that template somewhere. :-) I had to start from scratch. |
If I remember correctly, the base16-builder uses the newer scheme conventions (v0.8.0–0.9.1), which was added later on. The problem is that most Base16 schemes in the wild still use the older (original) convention (including many new schemes published independently). For this reason, in my project I opted to stick to the original scheme, even though it carries less information, so that I might use the classic Base16 schemes out of the box. Just be aware of this before switching tools and scheme versions. |
You're king of losing me a bit.
I don't know what "the base16-builder" is... there is a different builder for every language... but in theory they all do the same thing (follow the builder spec)... so perhaps you meant "a builder"? Your use of "the" just makes me wonder if you're talking about something else I don't know about.
I don't know what you mean here... to me the scheme conventions is just how the data is codified... where as what we really care about are the style conventions I would think... and I can't find any info on that prior to version 0.2. The builder conventions (the version numbers you reference) just describe how the builder itself outputs files, etc...
I'm afraid I truly have no idea what you mean when you say "older (original) convention" or "original scheme"... could you link me to something or be clear how an older scheme differs from a newer one if we're not talking about style conventions? |
The builder is an official tool, which supports different base16 versions: https://github.com/chriskempson/base16#builder-repositories These different Base16 schemes and builder versions differ in the way data is stored in the YAML/JSON file, where later versions use some different key naming conventions, and also introduce new key/values (e.g. other color representation beside the hex triplet, like RGB, etc.). It all boils down to were you'll be loading your base16 YAML/JSON scheme definitions, and whether they adopt the older or newer data convention. |
Yes, that's correct.
The base16-builder is a Ruby script (later replaced by a PHP script) that takes the schemes and applies them to a number of templates. Early versions of the builder contained templates mostly, but not exclusively, maintained by @chriskempson. Later on, he encouraged third-party templates to live in their own repositories, however they can be linked into base16-templates-source (the same goes for schemes) It's probably helpful to take a look at the Base16 eco-system. Let me also mention that there's a third-party Base16 builder for NodeJS.
That wasn't the point. I don't know if @chriskempson had his styleguide in mind when he started Base16 (and his own templates). I'm pretty sure that it didn't exist when I started making templates and schemes for it. It must have been around the time when the switch from the Ruby to the PHP builder was made, when the styleguide was made public. |
Well, I've never even heard of JSON for Base16... is that pre 0.8? (I only care about input format - obviously one could write a JSON template, though not sure why we'd care about that - we need CSS) Everything I've seen is YAML. I did review all the specs (from 0.8) onward and the only differences I see seem to be that the templates support more variables over time. In all cases the YAML scheme definitions remain exactly the same... I just forked and updated the Node builder, wrote a Highlight.js template and then used the builder to compile all schemes in https://github.com/chriskempson/base16-schemes-source. We use only
Just using |
I mean some of the schemes themselves are all over the map... but I don't know what we can do about that... no accounting for taste... and unless there is a prior style guide where the base#s used to have an alternative meaning (such that we could version control and build them differently) I'm not sure what could be done about that anyways? |
I'm not sure what you're directing me to exactly. I've read the website and the GitHub readme, and the style specs, the builder specs (0.8 onwards), the scheme specs, etc. I've written a template, written a scheme, forked and updated a builder. I think/hope I have a high level overview of things... I think perhaps I'm missing some nuance or history...? |
I don't think the schemes need to be questioned, I also don't think they have ever changed (there might have been additive changes). The real question is whether the Base16 template for highlight.js is a correct implementation that respects the style guideline. I'm talking about the template that existed since the early 2010s and that most themes are probably based on. If your own template follows the Base16 styleguide, I'd stick with that – even if that means breaking changes for some of the themes included in highlight.js. There is no point using the Base16 label for something that is not. The question then is, what happens to older themes that implement Base16 incorrectly. |
Well, I think it's a passable start for sure... though I'm open to feedback. :-) I posted a side-by-side over in the issue on the subject and we don't exactly match the sample (I'm not sure we should), but it's close. Obviously it's easy to update the template and rebuild the themes as improvements are made. OH. And boom, FINALLY understanding. :-) You're referring to https://github.com/chriskempson/base16-builder/tree/master/templates/highlight.js I had no idea such a thing existed as it's not documented anywhere nor included in |
Right that's my thinking. If "xyzzy" is a well known Base16 theme then if we include it it should have expected Base16 behavior - and be built using our Base16 template. The question I think becomes if we want to "augment" base16 somehow, but perhaps we just avoid doing that. And if older actual Base16 themes aren't very "base16-y" then that's their problem and they should be updated IMHO. :-)
Do you mean the included Highlight.js themes? They die. Unless the author/maintainer swears by their behavior in which case it gets more complex. :) If you mean older Base16 themes in the Base16 project then as I said above: if older actual Base16 themes aren't very "base16-y" then that's their problem and they should be updated IMHO. :-) |
Hi-sorry it's been a while, but yes I'm still interested |
Need to create a theme maintainers list so that theme maintainers who desire can keep their themes up-to-date.
The overall idea being that themes (like grammars) are never "one and done" that over time they require a (small) amount of maintenance to keep up with changes in the core library, grammars, theming in general, etc. Themes that have no owner/maintainer should possibly be moved into an
old_themes
repository/folder with a README that indicates they may not provide the best results compared to core themes.Of course we'd love it if all themes were maintained and provided a great experience.
What kind of things require upkeep? See #2500, #2521, etc... plus the recent additions of
property
,punctuation
,operator
, and other prior and upcoming changes... but this issue is just trying to get everyone onto a team so I can more easily contact you. After that I'll focus on typing up a list of things you may want to consider when refreshing your themes to take advantage of recent changes and improvements.Added by HLJS Team
Unmaintained
Base16
These should be auto-build from Base16 sources... see #3132.
Reached out
The text was updated successfully, but these errors were encountered: