From 2be309de1df5e4262175fbcc58e6dd8de4e0e6d3 Mon Sep 17 00:00:00 2001 From: JS Deck Date: Mon, 4 Aug 2025 16:29:39 -0300 Subject: virtualkeyboard: Add options to skip releasing pressed keys on close and to skip sharing key states (#11214) --- src/managers/input/InputManager.cpp | 92 +++++++++++++++++++++++++++++-------- src/managers/input/InputManager.hpp | 10 +++- 2 files changed, 81 insertions(+), 21 deletions(-) (limited to 'src/managers/input') diff --git a/src/managers/input/InputManager.cpp b/src/managers/input/InputManager.cpp index 6c23c493..cbb4c3cc 100644 --- a/src/managers/input/InputManager.cpp +++ b/src/managers/input/InputManager.cpp @@ -39,6 +39,7 @@ #include "../../managers/permissions/DynamicPermissionManager.hpp" #include "../../helpers/time/Time.hpp" +#include "../../helpers/MiscFunctions.hpp" #include @@ -931,6 +932,14 @@ Vector2D CInputManager::getMouseCoordsInternal() { return g_pPointerManager->position(); } +void CInputManager::newKeyboard(SP keeb) { + const auto PNEWKEYBOARD = m_keyboards.emplace_back(keeb); + + setupKeyboard(PNEWKEYBOARD); + + Debug::log(LOG, "New keyboard created, pointers Hypr: {:x}", (uintptr_t)PNEWKEYBOARD.get()); +} + void CInputManager::newKeyboard(SP keyboard) { const auto PNEWKEYBOARD = m_keyboards.emplace_back(CKeyboard::create(keyboard)); @@ -1398,18 +1407,36 @@ void CInputManager::onKeyboardKey(const IKeyboard::SKeyEvent& event, SPisVirtual() && shouldIgnoreVirtualKeyboard(pKeyboard); + const auto IME = m_relay.m_inputMethod.lock(); + const bool HASIME = IME && IME->hasGrab(); + const bool USEIME = HASIME && !DISALLOWACTION; + const auto EMAP = std::unordered_map{{"keyboard", pKeyboard}, {"event", event}}; EMIT_HOOK_EVENT_CANCELLABLE("keyPress", EMAP); - bool passEvent = DISALLOWACTION || g_pKeybindManager->onKeyEvent(event, pKeyboard); + bool passEvent = DISALLOWACTION; - if (passEvent) { - const auto IME = m_relay.m_inputMethod.lock(); + if (!DISALLOWACTION) + passEvent = g_pKeybindManager->onKeyEvent(event, pKeyboard); - if (IME && IME->hasGrab() && !DISALLOWACTION) { + if (passEvent) { + if (USEIME) { IME->setKeyboard(pKeyboard); IME->sendKey(event.timeMs, event.keycode, event.state); } else { + const auto PRESSED = shareKeyFromAllKBs(event.keycode, event.state == WL_KEYBOARD_KEY_STATE_PRESSED); + const auto CONTAINS = std::ranges::contains(m_pressed, event.keycode); + + if (CONTAINS && PRESSED) + return; + if (!CONTAINS && !PRESSED) + return; + + if (CONTAINS) + std::erase(m_pressed, event.keycode); + else + m_pressed.emplace_back(event.keycode); + g_pSeatManager->setKeyboard(pKeyboard); g_pSeatManager->sendKeyboardKey(event.timeMs, event.keycode, event.state); } @@ -1424,10 +1451,9 @@ void CInputManager::onKeyboardMod(SP pKeyboard) { const bool DISALLOWACTION = pKeyboard->isVirtual() && shouldIgnoreVirtualKeyboard(pKeyboard); - const auto ALLMODS = accumulateModsFromAllKBs(); - - auto MODS = pKeyboard->m_modifiersState; - MODS.depressed = ALLMODS; + auto MODS = pKeyboard->m_modifiersState; + const auto ALLMODS = shareModsFromAllKBs(MODS.depressed); + MODS.depressed = ALLMODS; const auto IME = m_relay.m_inputMethod.lock(); @@ -1437,6 +1463,7 @@ void CInputManager::onKeyboardMod(SP pKeyboard) { } else { g_pSeatManager->setKeyboard(pKeyboard); g_pSeatManager->sendKeyboardMods(MODS.depressed, MODS.latched, MODS.locked, MODS.group); + m_lastMods = MODS.depressed; } updateKeyboardsLeds(pKeyboard); @@ -1457,9 +1484,9 @@ bool CInputManager::shouldIgnoreVirtualKeyboard(SP pKeyboard) { if (!pKeyboard->isVirtual()) return false; - CVirtualKeyboard* vk = (CVirtualKeyboard*)pKeyboard.get(); + auto client = pKeyboard->getClient(); - return !pKeyboard || (!m_relay.m_inputMethod.expired() && m_relay.m_inputMethod->grabClient() == vk->getClient()); + return !pKeyboard || (client && !m_relay.m_inputMethod.expired() && m_relay.m_inputMethod->grabClient() == client); } void CInputManager::refocus() { @@ -1564,11 +1591,45 @@ void CInputManager::updateCapabilities() { m_capabilities = caps; } -uint32_t CInputManager::accumulateModsFromAllKBs() { +const std::vector& CInputManager::getKeysFromAllKBs() { + return m_pressed; +} + +uint32_t CInputManager::getModsFromAllKBs() { + return m_lastMods; +} + +bool CInputManager::shareKeyFromAllKBs(uint32_t key, bool pressed) { + bool finalState = pressed; - uint32_t finalMask = 0; + if (finalState) + return finalState; for (auto const& kb : m_keyboards) { + if (!kb->shareStates()) + continue; + + if (kb->isVirtual() && shouldIgnoreVirtualKeyboard(kb)) + continue; + + if (!kb->m_enabled) + continue; + + const bool PRESSED = kb->getPressed(key); + if (PRESSED) + return PRESSED; + } + + return finalState; +} + +uint32_t CInputManager::shareModsFromAllKBs(uint32_t depressed) { + uint32_t finalMask = depressed; + + for (auto const& kb : m_keyboards) { + if (!kb->shareStates()) + continue; + if (kb->isVirtual() && shouldIgnoreVirtualKeyboard(kb)) continue; @@ -1751,13 +1812,6 @@ void CInputManager::unsetCursorImage() { restoreCursorIconToApp(); } -std::string CInputManager::deviceNameToInternalString(std::string in) { - std::ranges::replace(in, ' ', '-'); - std::ranges::replace(in, '\n', '-'); - std::ranges::transform(in, in.begin(), ::tolower); - return in; -} - std::string CInputManager::getNameForNewDevice(std::string internalName) { auto proposedNewName = deviceNameToInternalString(internalName); diff --git a/src/managers/input/InputManager.hpp b/src/managers/input/InputManager.hpp index 1696dd0a..73a0dc48 100644 --- a/src/managers/input/InputManager.hpp +++ b/src/managers/input/InputManager.hpp @@ -94,6 +94,7 @@ class CInputManager { void onKeyboardKey(const IKeyboard::SKeyEvent&, SP); void onKeyboardMod(SP); + void newKeyboard(SP); void newKeyboard(SP); void newVirtualKeyboard(SP); void newMouse(SP); @@ -185,7 +186,8 @@ class CInputManager { CInputMethodRelay m_relay; // for shared mods - uint32_t accumulateModsFromAllKBs(); + const std::vector& getKeysFromAllKBs(); + uint32_t getModsFromAllKBs(); // for virtual keyboards: whether we should respect them as normal ones bool shouldIgnoreVirtualKeyboard(SP); @@ -194,7 +196,6 @@ class CInputManager { void setCursorImageUntilUnset(std::string); void unsetCursorImage(); - std::string deviceNameToInternalString(std::string); std::string getNameForNewDevice(std::string); void releaseAllMouseButtons(); @@ -305,6 +306,11 @@ class CInputManager { uint32_t accumulatedScroll = 0; } m_scrollWheelState; + bool shareKeyFromAllKBs(uint32_t key, bool pressed); + uint32_t shareModsFromAllKBs(uint32_t depressed); + std::vector m_pressed; + uint32_t m_lastMods = 0; + friend class CKeybindManager; friend class CWLSurface; }; -- cgit v1.2.3