On some task switcher themes, after Alt+Tab (and holding Alt) I can switch the tasks back and forward by pushing arrow keys. Such themes are "Cover Switch" and "Flip Switch". But on themes like "Informative" or "Compact" this is not possible. Reproducible: Always Steps to Reproduce: 1. Choose "Informative" as task switcher theme. Set Alt+Tab as a primary forward shortcut. 2. Open few random app windows. 3. Push Alt+Tab, to switch the tasks. While holding Alt, push Arrow Up key. (Nothing happens.) 4. Change the task switcher theme to "Flip Switch" and repeat the upper. (It works, the tasks are switched on Arrow key while holding Alt). Actual Results: Arrow keys work only on Flip Switch theme. Expected Results: Arrow keys to work on all available task switcher themes.
Unrelated to Task Manager, reassigning.
*** Bug 376623 has been marked as a duplicate of this bug. ***
*** Bug 374287 has been marked as a duplicate of this bug. ***
Can reproduce on also Wayland Cover Switch - Left/Right arrows are working, Up/Down arrows not working Flip Switch - All arrows are working Breeze - No arrows are working Breeze Dark - No arrows are working Grid - No arrows are working Compact - No arrows are working Small Icons - No arrows are working Text Only - No arrows are working Large Icons - No arrows are working Thumbnails - No arrows are working Informative - No arrows are working Operating System: KDE neon Developer Edition KDE Plasma Version: 5.14.80 Qt Version: 5.11.1 KDE Frameworks Version: 5.51.0 Kernel Version: 4.15.0-34-generic OS Type: 64-bit Processors: 8 × Intel® Core™ i5-8250U CPU @ 1.60GHz Memory: 7,7 GiB of RAM
*** Bug 351604 has been marked as a duplicate of this bug. ***
So basically everything that is "tabbox" (the popup windows and sidebar) don't support Arrow keys whereas all KWin effects (cover switch and flip switch) do.
Here's the links to what I found in Bug #374287 ------- It sounds like the QML based tabboxes aren't recieving the keypress events. The flip switch and cover switch are C++ desktop effects, which can hook the keyboard events themselves since they create their own window. https://github.com/KDE/kwin/blob/master/effects/flipswitch/flipswitch.cpp#L902 https://github.com/KDE/kwin/tree/master/effects/coverswitch This is where the QML based tabboxes events are handled. https://github.com/KDE/kwin/blob/master/tabbox/tabboxhandler.cpp#L618 If you do: PlasmaCore.Dialog { mainItem: Item { Keys.onPressed: {} } } then it works in the *preview* but does not work in the actual kwin popup. I wonder if it's because the popup isn't focused? PlasmaCore.Dialog is a QQuickWindow, which I don't think will do anything if it doesn't contain an activeFocusItem. https://github.com/qt/qtdeclarative/blob/dev/src/quick/items/qquickwindow.cpp#L1622 Did it work in the past? Did adding the eventFilter to support the mouse wheel break it? I don't remember if it used to work or not. https://github.com/KDE/kwin/commit/e9d20b80e9a79f514a219357d991092d83665350 I do know that the qml api changed, because a number of task switchers on GHNS no longer work.
The tabboxhandler.cpp eventFilter() function does not receive arrow key events: https://www.youtube.com/watch?v=noR582a0eBU
Some debug logging: kwin_tabbox: TabBox::keyPress 150994962 kwin_tabbox: TabBox::grabbedKeyEvent 16777234 kwin_tabbox: TabBoxHandler::grabbedKeyEvent 16777234 kwin_tabbox: d->m_mainItem && d->window() kwin_tabbox: d->window() PlasmaQuick::Dialog(0x15bcc30 exposed, visibility=QWindow::Visibility(Windowed), flags=QFlags<Qt::WindowType>(X11BypassWindowManagerHint|FramelessWindowHint), geometry=498,441 932x206) kwin_tabbox: d->window()->contentItem() QQuickRootItem(0x15820e0, parent=0x0, geometry=0,0 932x206) kwin_tabbox: d->window()->sendEvent Plasma::FrameSvgItem(0x15be890, parent=0x15820e0, geometry=0,0 932x206) kwin_tabbox: d->window()->sendEvent ColorScope(0x16862a0, parent=0x0, geometry=0,0 0x0) Okay, for some reason TabBoxHandler::grabbedKeyEvent is calling sendEvent to a ColorScope? We can easily find where it defines PlasmaCore.Dialog is the parent of the FrameSvgItem: https://github.com/KDE/plasma-framework/blob/master/src/plasmaquick/dialog.cpp#L758 The only other child of the contentItem() is the Dialog::mainItem. https://github.com/KDE/plasma-framework/blob/master/src/plasmaquick/dialog.cpp#L797 Wait, I just noticed the `ColorScope`'s `parent=0x0`, which means the Dialog is not the "parent"? Uhg. Does that mean it is not the mainItem()? I'm testing with my "Thumbnail Grid" skin, which is similar to the default skins in kdeplasma-addons/windowswitchers, Breeze's in plasma-workspace. * https://github.com/Zren/kwin-tabbox-thumbnail_grid/blob/master/package/contents/ui/main.qml * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/thumbnails/contents/ui/main.qml * https://github.com/KDE/plasma-workspace/blob/master/lookandfeel/contents/windowswitcher/WindowSwitcher.qml Actually why do we even care about the children of `contentItem()`?! Why not just iterate the children of the `window()` (the PlasmaCore.Dialog)? - const QList<QQuickItem*> items = d->window()->contentItem()->findChildren<QQuickItem*>(QString(), Qt::FindDirectChildrenOnly); + const QList<QQuickItem*> items = d->window()->findChildren<QQuickItem*>(QString(), Qt::FindDirectChildrenOnly); Aha! It works! https://www.youtube.com/watch?v=S8RL9NlHL8g
Here's a status of (most) QML Skins. * Breeze needs a key handler added. * Breeze Dark is falling back on Informative... which is weird. * United is falling back on a broken skin. I'll try deleting all the broken ones and check again. * The Informative skin probably just needs to reposition the Keys.onPressed inside the mainItem to work. ----- * Breeze (Look and Feel) * Source: plasma-workspace * KeyHandler: None * https://github.com/KDE/plasma-workspace/blob/master/lookandfeel/contents/windowswitcher/WindowSwitcher.qml * Breeze Dark (Look and Feel) * Source: breeze * Defined to use same as Breeze, but uses Informative. * KeyHandler: None * https://github.com/KDE/breeze/blob/master/lookandfeel.dark/contents/defaults#L17 * Thumbnail Grid * Source: https://store.kde.org/p/1153173/ * KeyHandler: Dialog.mainItem.Keys.onPressed * https://github.com/Zren/kwin-tabbox-thumbnail_grid/blob/master/package/contents/ui/main.qml * Preview Stack * Source: https://store.kde.org/p/1163050 * KeyHandler: None * Preview Reel * Source: https://store.kde.org/p/1161720 * KeyHandler: None * Small Icons * Source: kdeplasma-addons * KeyHandler: Dialog.mainItem.Keys.onPressed * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/small_icons/contents/ui/main.qml * Informative * Source: kdeplasma-addons * KeyHandler: Dialog.Keys.onPressed * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/informative/contents/ui/main.qml * Compact * Source: kdeplasma-addons * KeyHandler: Dialog.mainItem.Keys.onPressed * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/compact/contents/ui/main.qml * Thumbnails * Source: kdeplasma-addons * KeyHandler: Dialog.mainItem.Keys.onPressed * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/thumbnails/contents/ui/main.qml * Large Icons * Source: kdeplasma-addons * KeyHandler: Dialog.mainItem.Keys.onPressed * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/big_icons/contents/ui/main.qml * Text Only * Source: kdeplasma-addons * KeyHandler: Dialog.mainItem.Keys.onPressed * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/text/contents/ui/main.qml Broken: * United (Look and Feel) * Source: https://store.kde.org/p/1167950/ * Defined to use same as Breeze, but does not load. * https://github.com/llucassaw/United/blob/master/contents/defaults#L14 * Window Wall * Source: https://store.kde.org/p/1112559 * Broken (org.kde.kwin 0.1) * Black Icons * Source: https://store.kde.org/p/1112557 * Broken (org.kde.kwin ?) * present windows clone with a background * Source: https://store.kde.org/p/1112555 * Broken (org.kde.kwin 0.1) * Informative with thumbnail * Source: https://store.kde.org/p/1112556 * Broken (org.kde.kwin 0.1) * Present Windows Clone * Source: https://store.kde.org/p/1112558 * Broken (org.kde.kwin 0.1) * Scaling Switcher * Source: https://store.kde.org/p/1112560 * Broken (org.kde.kwin 0.1)
https://phabricator.kde.org/D15720
* Grid * Source: kdeplasma-addons * KeyHandler: Dialog.mainItem.Keys.onPressed * https://github.com/KDE/kdeplasma-addons/blob/master/windowswitchers/present_windows/contents/ui/main.qml ----- After removing all the broken skins, United is also defaulting to Informative. This is weird, it's not defaulting to the first/last alphabetically... My theory is that it's scanning for `org.kde.breeze.desktop`, doesn't find a kwin/tabbox/____ with that name, and picks the one before it (informative). Breeze Dark suffers this as well. big_icons compact informative <= Settles for the previous skin org.kde.breeze.desktop <= Doesn't exist in kwin/tabbox (it's lnf/windowswitcher) present_windows preview_reel preview_stack small_icons text thumbnail_grid thumbnails Breeze probably works since it contains the `windowswitcher/WindowSwitcher.qml` in it's LookAndFeel while the other 2 do not, so it thinks it's suppose to look in kwin/tabbox.
*** Bug 391339 has been marked as a duplicate of this bug. ***
Git commit c42d1607421e900567f6f4fe2644f18afa62e2f9 by David Edmundson. Committed on 07/11/2018 at 16:22. Pushed by davidedmundson into branch 'master'. [TabBox] Fix Arrow Key / Keyboard Events in QML Alt+Tab Skins Summary: KWin used to do a quirky trick to send key events to the topmost QQuickItem rather than things going to the activeFocus item. Sending it to the window previously would have failed as the window didn't think it was active. Since 66986d4afddcd09c28fe3addb0caa09279eda10f we can just let the window process the events in a normal QtQuick manner. Fixed-in: 5.15.0 It will require tabboxes to set focus correctly. The ones I tested did. Most analysis for this patch was done by Chris Holland. Test Plan: Held+alt tab with the "Text" tabbox switcher pressed up and down Reviewers: #kwin, graesslin Reviewed By: #kwin, graesslin Subscribers: graesslin, kwin, Zren Tags: #kwin Differential Revision: https://phabricator.kde.org/D16693 M +1 -7 tabbox/tabboxhandler.cpp https://commits.kde.org/kwin/c42d1607421e900567f6f4fe2644f18afa62e2f9
Git commit 4d52641657d691a860848fc1f724fe0a141676c9 by Nate Graham. Committed on 15/11/2018 at 14:02. Pushed by ngraham into branch 'master'. [windowswitcher] Implement keyboard navigation Summary: Implements keyboard navigation via the up and down arrow keys for the Breeze window switcher. Support for this went in with D16693. Test Plan: Alt-tab, use arrow keys, it works! Reviewers: #plasma, graesslin Reviewed By: #plasma, graesslin Subscribers: plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D16886 M +12 -0 lookandfeel/contents/windowswitcher/WindowSwitcher.qml https://commits.kde.org/plasma-workspace/4d52641657d691a860848fc1f724fe0a141676c9
Git commit 80bd32b12a01e29f3fe88f2a50c0a2a842872c19 by Chris Holland. Committed on 18/01/2019 at 17:44. Pushed by cholland into branch 'master'. Consistent arrow key handling in the Informative Alt+Tab skin Moved `Dialog.Keys.onPressed` to `Dialog.mainItem.Keys.onPressed` so that all tabbox skins are consistent. This fixes up/down arrow key navigation in this skin. Differential Revision: https://phabricator.kde.org/D16093 M +12 -11 windowswitchers/informative/contents/ui/main.qml https://commits.kde.org/kdeplasma-addons/80bd32b12a01e29f3fe88f2a50c0a2a842872c19
Is there a similar feature for the "Large Icons" switcher? Using Left/Right arrow keys there would make a lot of sense.
Yes, it works in 5.15.0.
I just figured out that for the normal task switching (Alt + Tab) the arrow keys work, but when switching between windows of the same application (Alt + Grave), the arrow keys do nothing... I think this is part of this same bug, but if you prefer, I can report another one.
*** Bug 404879 has been marked as a duplicate of this bug. ***
Git commit d7237df172e5a9065e387c94336cc38fd5c1fc13 by Vlad Zagorodniy, on behalf of Chris Holland. Committed on 05/03/2019 at 16:55. Pushed by vladz into branch 'Plasma/5.15'. Consistent arrow key handling in the Informative Alt+Tab skin Moved `Dialog.Keys.onPressed` to `Dialog.mainItem.Keys.onPressed` so that all tabbox skins are consistent. This fixes up/down arrow key navigation in this skin. Differential Revision: https://phabricator.kde.org/D16093 M +12 -11 windowswitchers/informative/contents/ui/main.qml https://commits.kde.org/kdeplasma-addons/d7237df172e5a9065e387c94336cc38fd5c1fc13
Created attachment 118915 [details] attachment-26989-0.html Thank you all, I see it working now. You rock! Иван Йонов http://i-creativ.net On Tue, 5 Mar 2019 at 18:57, Vlad Zagorodniy <bugzilla_noreply@kde.org> wrote: > https://bugs.kde.org/show_bug.cgi?id=370185 > > --- Comment #21 from Vlad Zagorodniy <vladzzag@gmail.com> --- > Git commit d7237df172e5a9065e387c94336cc38fd5c1fc13 by Vlad Zagorodniy, on > behalf of Chris Holland. > Committed on 05/03/2019 at 16:55. > Pushed by vladz into branch 'Plasma/5.15'. > > Consistent arrow key handling in the Informative Alt+Tab skin > > Moved `Dialog.Keys.onPressed` to `Dialog.mainItem.Keys.onPressed` so > that all tabbox skins are consistent. This fixes up/down arrow key > navigation in this skin. > > Differential Revision: https://phabricator.kde.org/D16093 > > M +12 -11 windowswitchers/informative/contents/ui/main.qml > > > https://commits.kde.org/kdeplasma-addons/d7237df172e5a9065e387c94336cc38fd5c1fc13 > > -- > You are receiving this mail because: > You reported the bug.