From a6d95ffdaf0b5f82e6309311baec1e49172994eb Mon Sep 17 00:00:00 2001
From: Damocles <damocles@darkest.space>
Date: Mon, 13 Apr 2026 15:36:54 +0200
Subject: [PATCH] notifications: derive popup enabled from maxPopups > 0

---
 modules/Modules.qml | 1 -
 nix/hm-module.nix   | 5 -----
 shell.qml           | 2 +-
 3 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/modules/Modules.qml b/modules/Modules.qml
index 3be68c1..3b5f0e9 100644
--- a/modules/Modules.qml
+++ b/modules/Modules.qml
@@ -22,7 +22,6 @@ QtObject {
     property var notifications: ({
             enable: true,
             timeout: 3000,
-            popups: true,
             maxPopups: 4
         })
     property var mpris: ({
diff --git a/nix/hm-module.nix b/nix/hm-module.nix
index a62a401..35a68e4 100644
--- a/nix/hm-module.nix
+++ b/nix/hm-module.nix
@@ -98,11 +98,6 @@ in
             default = 3000;
             description = "Notification popup timeout in milliseconds.";
           };
-          popups = lib.mkOption {
-            type = lib.types.bool;
-            default = true;
-            description = "Show notification popups.";
-          };
           maxPopups = lib.mkOption {
             type = lib.types.int;
             default = 4;
diff --git a/shell.qml b/shell.qml
index 7467ef3..3fc3674 100644
--- a/shell.qml
+++ b/shell.qml
@@ -20,7 +20,7 @@ ShellRoot {
             }
 
             LazyLoader {
-                active: Modules.notifications.popups
+                active: (Modules.notifications.maxPopups ?? 4) > 0
                 NotifPopup {
                     screen: scope.modelData
                 }