fix: code review fixes

- Fix hardcoded user path in webapps.nix (use homeDirectory)
- Normalize wallpapers option to use .enable suffix
- Remove duplicate FZF keybind declaration
- Update comments to match actual implementation
This commit is contained in:
m3tm3re
2026-04-26 10:49:01 +02:00
parent 1b5bcae686
commit 9908b9e335
6 changed files with 8 additions and 8 deletions

View File

@@ -1,7 +1,7 @@
# Web application desktop entries — Teams, Outlook, Basecamp, and OpenCode launchers.
{
config,
pkgs,
lib,
...
}: let
icons = {
@@ -42,7 +42,7 @@ in {
comment = "Open Basecamp as a Desktop App";
categories = ["Application" "Network"];
terminal = false;
icon = "/home/sascha.koenig/.local/share/icons/basecamp-logo.png";
icon = "${config.home.homeDirectory}/.local/share/icons/basecamp-logo.png";
};
opencode = {
name = "Opencode";

View File

@@ -5,9 +5,9 @@
...
}:
with lib; let
cfg = config.desktop.theme.wallpapers;
cfg = config.desktop.theme.wallpapers.enable;
in {
options.desktop.theme.wallpapers = mkEnableOption "wallpapers for Hyprland";
options.desktop.theme.wallpapers.enable = mkEnableOption "wallpapers for Hyprland";
config = mkIf cfg {
xdg.configFile."hypr/wallpapers" = {