diff options
author | vaxerski <[email protected]> | 2023-10-06 01:11:47 +0100 |
---|---|---|
committer | vaxerski <[email protected]> | 2023-10-06 01:11:47 +0100 |
commit | 4b3efc73c5d059e828f79952d80ff79c97521c3d (patch) | |
tree | a9559258388a6926b1352d223f5af75faee69d68 | |
parent | 24c04a8b7c874ead168bb2418f4325cb8bdaca5b (diff) | |
download | Hyprland-4b3efc73c5d059e828f79952d80ff79c97521c3d.tar.gz Hyprland-4b3efc73c5d059e828f79952d80ff79c97521c3d.zip |
events: fixup empty events not being sent on unmap
Ref #3506 https://github.com/hyprwm/Hyprland/issues/3506#issuecomment-1749508111
-rw-r--r-- | src/Compositor.cpp | 4 | ||||
-rw-r--r-- | src/events/Windows.cpp | 8 | ||||
-rw-r--r-- | src/managers/input/InputManager.cpp | 3 |
3 files changed, 12 insertions, 3 deletions
diff --git a/src/Compositor.cpp b/src/Compositor.cpp index 7fa45b07..66306888 100644 --- a/src/Compositor.cpp +++ b/src/Compositor.cpp @@ -893,6 +893,10 @@ void CCompositor::focusWindow(CWindow* pWindow, wlr_surface* pSurface) { g_pLayoutManager->getCurrentLayout()->bringWindowToTop(pWindow); if (!pWindow || !windowValidMapped(pWindow)) { + + if (!m_pLastWindow && !pWindow) + return; + const auto PLASTWINDOW = m_pLastWindow; m_pLastWindow = nullptr; diff --git a/src/events/Windows.cpp b/src/events/Windows.cpp index 54ffb37e..f78a2975 100644 --- a/src/events/Windows.cpp +++ b/src/events/Windows.cpp @@ -737,8 +737,14 @@ void Events::listener_unmapWindow(void* owner, void* data) { g_pInputManager->simulateMouseMovement(); else g_pCompositor->focusWindow(PWINDOWCANDIDATE); - } else { + } else g_pInputManager->simulateMouseMovement(); + + // CWindow::onUnmap will remove this window's active status, but we can't really do it above. + if (PWINDOW == g_pCompositor->m_pLastWindow || !g_pCompositor->m_pLastWindow) { + g_pEventManager->postEvent(SHyprIPCEvent{"activewindow", ","}); + g_pEventManager->postEvent(SHyprIPCEvent{"activewindowv2", ","}); + EMIT_HOOK_EVENT("activeWindow", (CWindow*)nullptr); } } else { Debug::log(LOG, "Unmapped was not focused, ignoring a refocus."); diff --git a/src/managers/input/InputManager.cpp b/src/managers/input/InputManager.cpp index fe66b8ba..1b89f8fc 100644 --- a/src/managers/input/InputManager.cpp +++ b/src/managers/input/InputManager.cpp @@ -314,9 +314,8 @@ void CInputManager::mouseMoveUnified(uint32_t time, bool refocus) { wlr_seat_pointer_clear_focus(g_pCompositor->m_sSeat.seat); - if (refocus) { // if we are forcing a refocus, and we don't find a surface, clear the kb focus too! + if (refocus || !g_pCompositor->m_pLastWindow) // if we are forcing a refocus, and we don't find a surface, clear the kb focus too! g_pCompositor->focusWindow(nullptr); - } return; } |