-
Notifications
You must be signed in to change notification settings - Fork 2
Description
Is legacy/original base16 support a focus or not for tinted-builder-rust? If no, reject this issue :-) if yes, see below for enhancement ideas
As an experiment, I tried building using https://github.com/stonebuddha/base16-chinoiserie-scheme. This fails due to tinted-builder-rust expecting a system which https://github.com/chriskempson/base16/blob/main/styling.md does not list at all, and neither does https://github.com/tinted-theming/home/blob/main/styling.md#legacy-ayu-darkyaml
error:
Error: Unable to deserialize scheme "C:\Users\clach04\AppData\Roaming\tinted-theming/tinted-builder-rust\schemes\base16\chinoiserie.yaml": missing field `system`
Idea, if system is missing default to base16
If system is manually added, then get a name lookup failure, again https://github.com/chriskempson/base16/blob/main/styling.md doesn't mention this. Interestingly this scheme uses the field scheme for the name, I'm not clear is this is an old bsae16 standard or just a bad scheme
Error: Unable to deserialize scheme "C:\Users\clach04\AppData\Roaming\tinted-theming/tinted-builder-rust\schemes\base16\chinoiserie_tinted.yaml": missing field `name`
Idea if name is missing, using slug (if present) or filename.
Then if those fields are present get an error about the pallete:
Error: Unable to deserialize scheme "C:\Users\clach04\AppData\Roaming\tinted-theming/tinted-builder-rust\schemes\base16\chinoiserie_tinted.yaml": base16 scheme does not contain the required palette properties
Manually adding pallete and indenting the colors then works.