rope/nixos
1
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

quickshell: drop the focus grab before activating a notification's window

Browse source 
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
rope 2026-08-03 10:39:53 +01:00
parent 3d5ecb7b98
commit 99525c98ed
1 changed files with 17 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

19
settings/quickshell.nix
View file

@ -1483,7 +1483,15 @@ in
// Invoking an action ("View", "Reply"…) // Invoking an action ("View", "Reply"…)
// makes the app want focus, but it can't // makes the app want focus, but it can't
// take it itself — see focusNotifApp. // take it itself — see focusNotifApp.
onClicked: { modelData.invoke(); bar.focusNotifApp(_nc.notif); _nc.actionInvoked(); } // closeAllDropdowns first for the same
// focus-grab reason as the list card; it's
// a no-op for the toast, which has no grab.
onClicked: {
bar.closeAllDropdowns();
modelData.invoke();
bar.focusNotifApp(_nc.notif);
_nc.actionInvoked();
}
} }
} }
} }
@ -6084,10 +6092,17 @@ in
MouseArea { MouseArea {
anchors.fill: parent anchors.fill: parent
cursorShape: Qt.PointingHandCursor cursorShape: Qt.PointingHandCursor
// Close FIRST: the panel holds a
// HyprlandFocusGrab, and tearing that
// down refocuses whatever was focused
// before — which undoes our activate if
// it already ran. animateClose() drops
// activeDropdown synchronously, so the
// grab is gone before the next line.
onClicked: { onClicked: {
bar.closeAllDropdowns();
bar.activateNotif(notifItem.modelData); bar.activateNotif(notifItem.modelData);
notifItem.modelData.dismiss(); notifItem.modelData.dismiss();
bar.closeAllDropdowns();
} }
} }