Remove redundant shell escaping and string coercion as stylix.image is
guaranteed to be a stringified store path with a valid string context
and no special characters.
Fixes: 3499e3ec70 ("treewide: properly quote stylix.image when used as a shell argument")
Optionalize mkTarget's 'humanName' and 'name' arguments by inferring
'humanName' from the 'name' attribute in the /modules/<MODULE>/meta.nix
file, and 'name' from the /modules/<NAME>/ directory name.
Inferring the 'humanName' and 'name' arguments ensures consistency and
reduces boilerplate.
The 'humanName' and 'name' arguments are optionalized instead of removed
because complex modules generating target derivations need to
distinguish between them.
Closes: https://github.com/nix-community/stylix/issues/1661
Rename mkTarget's 'configElements' argument to 'config' and
'extraOptions' to 'options' to provide a more transparent interface with
the underlying Nixpkgs module system.
Optionalize the stylix.image option when stylix.base16Scheme is set,
making the following Stylix configurations valid:
[
// Now it possible to set 'stylix.image = null', if
// stylix.base16Scheme is set.
{
base16Scheme = /* ... */;
}
// This configuration was already possible.
{
image = /* ... */;
}
// This configuration was already possible.
{
base16Scheme = /* ... */;
image = /* ... */;
}
]
Closes: https://github.com/danth/stylix/issues/200
Closes: https://github.com/danth/stylix/issues/442
Link: https://github.com/danth/stylix/pull/717
Co-authored-by: NAHO <90870942+trueNAHO@users.noreply.github.com>
Reviewed-by: NAHO <90870942+trueNAHO@users.noreply.github.com>
Tested-by: NAHO <90870942+trueNAHO@users.noreply.github.com>
Reviewed-by: Daniel Thwaites <danth@danth.me>
Initialize the 'hyprpaper' module.
In the future, it might be possible to 'unload' the one-off wallpaper to reduce
the running memory usage [1]:
> Preload does exactly what it says. It loads the entire wallpaper into
> memory. This can result in around 8 - 20MB of mem usage. It is not
> recommended to preload every wallpaper you have, as it will be a)
> taking a couple seconds at the beginning to load and b) take 100s of
> MBs of disk and RAM usage.
>
> Preload is meant only for situations in which you want a wallpaper to
> switch INSTANTLY when you issue a wallpaper keyword (e.g. wallpaper
> per workspace)
>
> In any and all cases when you don't mind waiting 300ms for the
> wallpaper to change, consider making a script that:
> - preloads the new wallpaper
> - sets the new wallpaper
> - unloads the old wallpaper (to free memory)
>
> -- Hyprpaper [2]
[1]: https://github.com/danth/stylix/issues/408
[2]: 678d0e8959/README.md (L103-L111)
Co-authored-by: NAHO <90870942+trueNAHO@users.noreply.github.com>
