-
Notifications
You must be signed in to change notification settings - Fork 28
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
Inconsistent shade spanning for some schemes #8
Comments
It seems that this rule is only an advice. There are multiple themes that don't follow it and use colors in unorthodox ways. |
I thought about asking this question in a separate thread or asking with a commit of a new topic, but I'll ask here anyway. Base16 is about the specification (unification)? I think so. I also noticed that some color schemes do not maintain a consistent style when composing fields: The same is true for the author field (there is a name somewhere and a link somewhere, an email) Maybe it’s worth considering a unified design style when all attributes must be present in the .yaml description:
meta:
What can this actually do? Well, if you use any builders, you can be sure that when iterating through the fields you will not get an exception that a key is missing. If it's really useful, I can take the time to rework the schemes. |
I meant to do this when we switched to a separate schemes repo and updated the spec. I'll look at doing this soon. For what it's worth, the original reason for not requiring the |
In my mind, there are a few parts of this - consistent metadata and consistent colors. Consistent Metadata
For missing fields, we've done it this way to stay as backwards compatible as possible and allow older builders to build newer schemes. However, with the direction we're going in, making more fields required might be an option. The current spec is defined in the home repo. Slug in particular is essentially a workaround for a workaround. Pre-fork the filename was used as the scheme's name, however we wanted to make each scheme file self-contained, so we defined a way to "slugify" the name. Unfortunately, that resulted in a few output files being renamed, so we added the "slug" field as a way to override the "slugify" method of determining the scheme name. I'd love to deprecate this and move to requiring the slug. I've filed an issue to this effect. Consistent ColorsFor consistent colors (which I think is what you called If you have specific schemes you like which you think you can fix (as mentioned in the original comment, tokyodark is particularly egregious), we're open to merging improvements to existing schemes. |
Also related to consistent colors: there have been a number of proposals around how to better handle styling in base16 theme templates. |
We've added clearer information to the styling spec, so while it still doesn't absolutely require the suggested colors, it's at least clearly suggested. As for situations like the color disparity in tokyonight, I think we can adjust these schemes to more closely match the styling spec. In the case of tokyonight, it's quite easy to go to non base16/24 themes and see that they contain red,yellow,green,cyan,etc and put them in the suggested places. @MOIS3Y what ideas did you have for the meta property? |
It looks like that most of the
tokyonight-*
schemes breaks this rule, creating inconsistent coloring.tokyodark.yaml
is a particularly egregious example:The colors
base08
-base0F
is also kind of random. There are probably other non-compliant themes, but these are the ones I found.Tbh, I wish that the spec makes it so that the color values are prefixed with a
#
, to make it easier to inspect and edit in editors that supports color highlighting/manipulation.The text was updated successfully, but these errors were encountered: