punkfairie/rose-pine-nix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(home-manager): assert qt.style.name for kvantum theme (#242)

Browse source 
while having `qt.enable` be a prerequisite for this option applying
makes sense, the requirement on a specific `style.name` is not. this
make it more clear by moving that check to an assertion rather than an
internal comparison
This commit is contained in:
seth 2024-06-25 10:29:50 +00:00 committed by GitHub
parent 6e77fdd91d
commit 1adbfeb44a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 21 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
modules/home-manager/kvantum.nix
View file

@ -6,13 +6,8 @@
}:
let
cfg = config.qt.style.catppuccin;
enable =
cfg.enable
&& config.qt.enable
&& lib.elem config.qt.style.name [
"kvantum"
"Kvantum"
];
enable = cfg.enable && config.qt.enable;
flavorCapitalized = lib.ctp.mkUpper cfg.flavor;
accentCapitalized = lib.ctp.mkUpper cfg.accent;
theme = pkgs.catppuccin-kvantum.override {
@ -35,7 +30,18 @@ in
};
};
config.xdg.configFile = lib.mkIf enable {
config = lib.mkIf enable {
assertions = [
{
assertion = lib.elem config.qt.style.name [
"kvantum"
"Kvantum"
];
message = ''`qt.style.name` must be `"kvantum"` to use `qt.style.catppuccin`'';
}
];
xdg.configFile = {
"Kvantum/${themeName}".source = "${theme}/share/Kvantum/${themeName}";
"Kvantum/kvantum.kvconfig" = lib.mkIf cfg.apply {
text = ''
@ -44,4 +50,5 @@ in
'';
};
};
};
}