Bug 397210 - Touchpad applet is missing from the system tray under Wayland
Summary: Touchpad applet is missing from the system tray under Wayland
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Touchpad (show other bugs)
Version: 5.21.2
Platform: Neon Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
: 407582 (view as bug list)
Depends on:
Blocks:
 
Reported: 2018-08-06 12:55 UTC by Patrick Silva
Modified: 2021-05-18 18:16 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.23


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2018-08-06 12:55:42 UTC
"Touchpad" is checked in the system tray settings under "Extra items" but its icon only appears in the system tray under X11.

I use neon dev unstable.
Comment 1 Alexander Mentyu 2018-08-27 16:33:09 UTC
Can confirm

Distribution: KDE neon Developer Edition
Kernel: 4.15.0-29-generic
Plasma: 5.13.80
Apps: 18.11.70
Qt: 5.11.1
Frameworks: 5.48.0
Graphics:
Card-1: Intel driver: i915 v: kernel bus ID: 00:02.0 chip ID: 8086:5917 
Card-2: NVIDIA GM108M [GeForce 940MX] driver: nouveau v: kernel bus ID: 01:00.0 chip ID: 10de:134d 
Display: wayland server: X.Org 1.19.6 driver: modesetting,nouveau 
unloaded: fbdev,vesa alternate: nvidia compositor: kwin wayland 
resolution: 1920x1080~57Hz 
OpenGL: renderer: Mesa DRI Intel UHD Graphics 620 (Kabylake GT2) v: 4.5 Mesa 18.0.5 
compat-v: 3.0 direct render: Yes
Comment 2 Patrick Silva 2019-03-17 18:43:24 UTC
bug persists

Operating System: KDE neon Developer Edition
KDE Plasma Version: 5.15.80
KDE Frameworks Version: 5.57.0
Qt Version: 5.12.0
Comment 3 Patrick Silva 2019-05-17 11:33:39 UTC
*** Bug 407582 has been marked as a duplicate of this bug. ***
Comment 4 David Edmundson 2020-01-22 22:02:50 UTC
Urgh, the dataengine code reimplements talking to the DBus interface, but without the helpful abstraction layer.

Should be a relatively easy fix.
Comment 5 David Edmundson 2020-01-26 19:06:54 UTC
Touchpad has 3 backends, 2 for X one for wayland.

The kded handles global shortcuts and is a proxy for data for the applet.
It supports talking to all backends

The problem is that on wayland: 
 - the relevant global shortcuts were moving into kwin directly
 - so the kded module was disabled on wayland to avoid a conflict
 - but the applet still tries to talk to kded
 - relevant information is available for kwin, but in a different way

We can: 
 - put an abstraction layer in the plasmoid to speak to kded or kwin
 - make kwin expose the same interface as kded
 - re-enable the kded on wayland and remove shortcut handling from kwin

I don't know which is best
Comment 6 Bug Janitor Service 2021-05-07 20:56:15 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/455
Comment 7 Nate Graham 2021-05-18 18:16:32 UTC
Git commit 3b4026641454c02ad3030c8cd2134a672995ffa9 by Nate Graham.
Committed on 18/05/2021 at 18:15.
Pushed by ngraham into branch 'master'.

Delete the Touchpad applet

Ths only thing this applet can do is disable or re-enable your
touchpad. If you have no touchpad, it is pointless. If you have only a
touchpad and no mouse, then the only thing it can do is give you a way
to brick your laptop until you find a mouse you can use to re-enable
the touchpad.

The only conceivable non-catastrophic usage of this applet is to
manually disable a laptop's touchpad when a mouse is plugged in. This is
an extremely niche use case that can already be accomplished by using
the "Toggle Touchpad" keyboard shortcut which is exported by the KCM,
and would be better served by a setting in the touchpad KCM itself to
automatically disable the touchpad when a mouse is plugged in and
re-enable it when unplugged, should such a feature be deemed useful. I
have my doubts, as Libinput has gotten very good at ignoring false input
these days, so I think there is no longer much of a use case for
disabling a touchpad at all.

Overall, this applet is not worth the bugs and degradation of the user
experience that it can cause. It is too dangerous to exist. Let's delete
it.
Related: bug 434720, bug 434718, bug 378879, bug 430029, bug 434717
FIXED-IN: 5.23

M  +1    -1    kcms/touchpad/CMakeLists.txt
D  +0    -155  kcms/touchpad/applet/qml/contents/ui/touchpad.qml
D  +0    -160  kcms/touchpad/applet/qml/metadata.desktop
D  +0    -30   kcms/touchpad/applet/touchpad.svg
R  +4    -12   kcms/touchpad/dataengine/CMakeLists.txt [from: kcms/touchpad/applet/CMakeLists.txt - 065% similarity]
R  +0    -0    kcms/touchpad/dataengine/plasma-dataengine-touchpad.desktop [from: kcms/touchpad/applet/plasma-dataengine-touchpad.desktop - 100% similarity]
R  +0    -0    kcms/touchpad/dataengine/touchpad.operations [from: kcms/touchpad/applet/touchpad.operations - 100% similarity]
R  +0    -0    kcms/touchpad/dataengine/touchpadengine.cpp [from: kcms/touchpad/applet/touchpadengine.cpp - 100% similarity]
R  +0    -0    kcms/touchpad/dataengine/touchpadengine.h [from: kcms/touchpad/applet/touchpadengine.h - 100% similarity]
R  +0    -0    kcms/touchpad/dataengine/touchpadservice.cpp [from: kcms/touchpad/applet/touchpadservice.cpp - 100% similarity]
R  +0    -0    kcms/touchpad/dataengine/touchpadservice.h [from: kcms/touchpad/applet/touchpadservice.h - 100% similarity]

https://invent.kde.org/plasma/plasma-desktop/commit/3b4026641454c02ad3030c8cd2134a672995ffa9