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

kde: conflict with plasma-manager #250

Closed
Noodlez1232 opened this issue Feb 6, 2024 · 3 comments
Closed

kde: conflict with plasma-manager #250

Noodlez1232 opened this issue Feb 6, 2024 · 3 comments
Labels
bug Something isn't working properly

Comments

@Noodlez1232
Copy link
Contributor

The plasma-manager home-manager "extension" overwrites some of the files that are used by stylix (specifically "kcminputrc", "kded5rc", and "kdeglobals").
At first I thought of opening up the issue there, but I think it'd be worthwhile to look into using this to do our KDE management instead, since it might be easier later on to add options to stylix (and also compatibility because I could see other people using both at the same time).

This may also be related to pull request #142, since its inclusion would remove the need for "kdeglobals"

@danth danth added the bug Something isn't working properly label Feb 7, 2024
@Noodlez1232
Copy link
Contributor Author

I'm actually working on this now, but I was wondering if there were to be any interest in this if I were to open up a pull request, or would it be not needed?
I think one of the main problems is that plasma-manager makes an option called "plasma.enable" that we could possibly use for stylix's mkEnableTarget.

@Noodlez1232
Copy link
Contributor Author

As it turns out, after doing some more poking around, this is a bug with my configuration not with stylix. I'm closing it, but I will probably open another one with the possibility of using plasma-manager still, since I think it's still useful to avoid some of the messiness of the module. For now I'll start my improvements there, and then probably come here again with some suggestions on that.

@danth
Copy link
Owner

danth commented Mar 20, 2024

If we replaced the module with plasma-manager, then everyone would be required to have that installed to get theming. We could try something similar to the situation with NeoVim, where there are separate targets for the built in Home Manager module and NixVim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working properly
Projects
None yet
Development

No branches or pull requests

2 participants