aboutsummaryrefslogtreecommitdiffstats
path: root/notifications/NotificationBalloon.qml
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-09-15 14:54:36 +0200
committerDanilo M. <danix@danix.xyz>2026-09-15 14:54:36 +0200
commitc0473d87565569202d548eba47279321e287990c (patch)
tree2f869d1330da836addd8e373a81ee2e96ac3c376 /notifications/NotificationBalloon.qml
parent708eeda25d3d6e47afaa75fbb77a162cb3c1ff09 (diff)
downloadquickshell-c0473d87565569202d548eba47279321e287990c.tar.gz
quickshell-c0473d87565569202d548eba47279321e287990c.zip
fix(notifications): apply final review fixes
Mirror the balloon's right-click close-all on the drawer row: the spec says the gestures are identical in both forms, and a right-click on a row previously did nothing. The right-button branch precedes the live guard so it behaves the same on history rows. Stop importing the Status singleton into the notification shell. Referencing it instantiated it, and its onPresentationChanged writes status.dnd and runs breaktimer.sh, so a read-only consumer was writing state and shelling out on every presentation toggle, and doubled the dndBeforePresentation race. Notify now reads "/run/user/<uid>/status.dnd" directly through a FileView, the same convention as the notifyd files; a missing file means off. AGENTS.md named Drawer.qml as the drawer's reserved space; it is desktop/NotificationList.qml.
Diffstat (limited to 'notifications/NotificationBalloon.qml')
-rw-r--r--notifications/NotificationBalloon.qml2
1 files changed, 1 insertions, 1 deletions
diff --git a/notifications/NotificationBalloon.qml b/notifications/NotificationBalloon.qml
index e3763da..b8ce0f7 100644
--- a/notifications/NotificationBalloon.qml
+++ b/notifications/NotificationBalloon.qml
@@ -31,7 +31,7 @@ Rectangle {
visible: {
if (Notify.drawerOpen) return false;
if (Notify.snoozeUntil > b.now) return false;
- if (Status.dnd && b.notification.urgency !== "critical") return false;
+ if (Notify.dnd && b.notification.urgency !== "critical") return false;
if (b.notification.expires !== 0 && b.now >= b.notification.expires) return false;
return true;
}