Skip to content
This repository has been archived by the owner on Dec 16, 2023. It is now read-only.

fix color scheme names #4

Merged
merged 1 commit into from
Jul 3, 2022
Merged

Conversation

baodrate
Copy link
Contributor

Fix names in the schemes (incorrect or duplicate names)

Some slight inconsistencies still exist (not sure if I should touch these or not):

  • some file names are joined rather than hyphenated (where the name specified in the scheme is separated by spaces)
    • (blueforest, brushtrees, darkviolet, greenscreen, irblack, solarflare, windows-highcontrast)
  • some file names are missing a portion of the name specified in the scheme
    • (Heetch Dark, Icy Dark, Seti UI, Synth Midnight Terminal, London Tube)

@belak
Copy link
Member

belak commented Jun 26, 2022

Thanks for doing this! Would you be willing to rename those schemes to match rather than renaming the files (for horizon-light and horizon-dark)?

@baodrate
Copy link
Contributor Author

The horizon-terminal- schemes' files were not renamed (there are separate horizon- schemes). I assume you mean the harmonic16 schemes?

Are you sure we want to rename those schemes? harmonic16 is the name the repo is under, as well as the value of scheme in the base16 scheme file generated by the Harmonic16 generator.

And what about the other schemes I mentioned in the description (e.g. Seti UI). Should I fix the filename, the scheme file, or leave it alone?

@belak
Copy link
Member

belak commented Jun 26, 2022

Good points.

Yes, I meant the harmonic16 schemes, but I think you're right that they should be renamed. I didn't realize they came from something called harmonic16 (I assumed it was harmonic and they added the 16 because it was base16).

I'll think on the others. The issue with renaming the files is that it changes the slugs and downstream will change the names of the schemes people load.

@baodrate
Copy link
Contributor Author

good point, it seems reasonable to avoid changing the filenames if it'll affect downstream users

I did some quick scripting with yq and the lists specified in the OP should be comprehensive. All the schemes not addressed by this PR seem to have matching filenames and display names. If you are OK with the renaming of harmonic16, feel free to merge this PR. If you'd still prefer to avoid renaming that file, I can amend the commit

@belak
Copy link
Member

belak commented Jul 3, 2022

Sorry for the delay - this looks good to me! Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants