themes icon indicating copy to clipboard operation
themes copied to clipboard

feat: update theme schema

Open fedeci opened this issue 2 years ago • 3 comments

For backward compatibility we could have a duplicated theme property to point to the default settings. This should be deprecated after one or two versions. We currently fallback to the default light theme which is embedded in the engine so autocomplete would still work without the themes.

fedeci avatar Jan 04 '22 20:01 fedeci

With this new format we need to add an action to recreate the settings file in macOS repo each time a new config is added or edited. (Similar to what we did for API docs but way simpler)

fedeci avatar Jan 08 '22 10:01 fedeci

Hmmm - want to hear what the other teammates have to say, however, I personally don't think making variants per theme makes too much sense. I think a theme should just be one thing (e.g. light OR dark) and the system theme essentially switches you from one theme to another (rather than a variant within one theme). Then we can eventually let users choose a daytime mode theme and a nighttime mode theme.

I think we are going to start using this themes repo pretty heavily. Not just for autocomplete themes but for mission control themes, terminal themes etc. I can imagine there being hundreds or thousands of themes in the future and almost all of these will just be one variant.

Summary: I agree the schema for themes should be re-imagined, however, I don't believe it should variants makes too much sense..

brendanfalk avatar Jan 10 '22 23:01 brendanfalk

I partially agree with you, my intention with these changes was actually just to add light and dark under variants and eventually an high contrast trying to emulate what vscode is doing. Currently we do not allow theme creators to have both a dark and light version all in one place. While working on system theme I thought it would have been cool to also allow users to create themes that toggle automatically between light and dark versions. My intention is to provide the setting to configure variants if and only if variants has more than one theme. So the experience for users would not be radically changed. For sure wanna hear more opinions because this is a super important change that won't probably allow other breaking updates in the future.

fedeci avatar Jan 11 '22 19:01 fedeci