https://bugs.kde.org/show_bug.cgi?id=448953

Jakob Petsovits <jpe...@petsovits.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/plasma-desktop/-/commit/ |ma/plasma-desktop/-/commit/
                   |295b8e616d5bc9fb8604d492fc5 |10b6daa59e86bdb60ea29e92780
                   |3b5a4eeaeb1c1               |2673f453fecbf

--- Comment #15 from Jakob Petsovits <jpe...@petsovits.com> ---
Git commit 10b6daa59e86bdb60ea29e927802673f453fecbf by Jakob Petsovits.
Committed on 24/01/2025 at 06:36.
Pushed by jpetso into branch 'Plasma/6.3'.

kcms/touchpad: Make sure settings fields in X11 are not randomly missing

We will now load X11 props early for constants like "supports $xyz".

Before this, if QML makes use of a property during initialization,
it may incorrectly assume "unsupported" for a given libinput feature
and subsequent "load" calls won't fix that value, as its `Q_PROPERTY`
declaration marks it as `CONSTANT`.

So let's move anything into the device (LibinputTouchpad) constructor
that determines a `Q_PROPERTY` without change notifications.

A number of accessors for "supported" flags are simplified because
their .val boolean is sufficient by itself, remaining at their
default `false` value if the feature is unavailable.


(cherry picked from commit 295b8e616d5bc9fb8604d492fc53b5a4eeaeb1c1)

Co-authored-by: Jakob Petsovits <jpe...@petsovits.com>

M  +43   -31   kcms/touchpad/backends/x11/libinputtouchpad.cpp
M  +10   -10   kcms/touchpad/backends/x11/libinputtouchpad.h

https://invent.kde.org/plasma/plasma-desktop/-/commit/10b6daa59e86bdb60ea29e927802673f453fecbf

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to