Skip to content
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

Kotlin Rewrite: add accent colours other than mauve #58

Open
FelixBrakel opened this issue Mar 14, 2023 · 8 comments
Open

Kotlin Rewrite: add accent colours other than mauve #58

FelixBrakel opened this issue Mar 14, 2023 · 8 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@FelixBrakel
Copy link

Love the theme, my entire system is catppuccin now.

One thing is that I couldn't find a way to change the eye candy color, I really like peach. Does the theme support it?

The plugin page does mention it, but I think that's just because that is the standard catppuccin description that was copied over?

Consider updating the description. (Or adding support for the different eye candy colors)

@sgoudham
Copy link
Contributor

Heya @FelixBrakel, the plugin, unfortunately, doesn't support all the eye-candy/accent colours atm. You're right that the description is copied over from the main repository.

In order to support the recompilation on the go, I believe this repository will need a Kotlin rewrite. Something that I don't have the time to do at the minute. I think this should be tracked on GitHub so, if it's alright with you, I can turn this into a tracking issue to support all the different eye candy colours!

@nekowinston
Copy link
Contributor

Do you mean the accent colors, similar to VSCode?
Eye-candy here just refers to the colors being pleasant to look at, copied from the main repo description.

We'd need to rewrite the current theme with compilation support in Kotlin, so that when the user changes the accent color, the XML is regenerated. Both @sgoudham and I don't feel comfortable enough with Kotlin to attempt this - we had offers from people wanting to rewrite it, but nothing has come to fruition yet.

@FelixBrakel
Copy link
Author

Not having time is just a fact of life when it comes to community projects so no worries, I was just trying to clarify.

It's not like I have the time to do dive into Kotlin and the development of this theme at the moment either.

@sgoudham sgoudham added the help wanted Extra attention is needed label Mar 14, 2023
@sgoudham sgoudham changed the title Eye candy colors Kotlin Rewrite: add accent colours other than mauve Mar 14, 2023
@sgoudham sgoudham added the enhancement New feature or request label Mar 14, 2023
@marnym
Copy link

marnym commented Jul 21, 2023

I have made an attempt on solving this.

The problem I have encountered is how to re-generate the files on accent color selection. Since the theme files are in the resources directory they cannot be modified. It might be impossible to re-generate the files as long as the theme files are in the resources directory.

Another non-optimal solution would be to just generate all variants at build time, but this will create a lot of unwanted options in the theme-selection drop down.

@FelixBrakel
Copy link
Author

The latter is the solution many other implementations of this theme are going for, I wouldn't mind it.

@sgoudham
Copy link
Contributor

sgoudham commented Oct 2, 2023

Hi @FelixBrakel @isofore, just revisiting this issue based on #92 (cc: @JosephAbbey)

I didn't take this into account at the time but if we were to generate XML's for each flavour, accent colour and italic/non-italic. That's actually 4 * 14 * 2 which is a massive 112 XML files to ship. This would severely impact people's ability to scroll through other themes that they have installed so, unfortunately, I don't think we can go ahead with that.

Going off the discussion within the discord, there may be scope in potentially uploading multiple versions of the plugin on the marketplace, resulting in 14 jars for each account colour. I'm currently unsure if there's a marketplace policy that prevents this but it might be our best bet at shipping different UI colours without diving into the Kotlin Rewrite.

We could limit the accent coloured jars to just GitHub Releases but the tradeoff is that users will have to install plugins from disk, and therefore will not receive updates from the marketplace for that plugin, meaning that the user is in control of managing any updates (which I'm not a fan of)

I'm interested in what others think? Honestly, I'm swaying more towards just putting effort into a Kotlin rewrite so that we're following JetBrains' solution to reloading the UI.

@JosephAbbey
Copy link

I think that the kotlin approach is the way forward, because it would actually be 4 * 14 * 14 * 2 if selecting a secondary accent colour is allowed (currently defaults to yellow).

@marnym
Copy link

marnym commented Oct 3, 2023

The theme generation in Kotlin should be done. It was the regeneration on the fly which I never managed to do. Link to the code: https://github.com/isofore/jetbrains/tree/accent-colors.

Hopefully some of it would be beneficial to whomever might do the rewrite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants