https://bugs.kde.org/show_bug.cgi?id=402634
Nicolas Fella <nicolas.fe...@gmx.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://invent.kde.org/util
| |ities/yakuake/commit/f0a2c7
| |2a6c52a16a0f74382e3c5d3218d
| |aa4363d
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #13 from Nicolas Fella <nicolas.fe...@gmx.de> ---
Git commit f0a2c72a6c52a16a0f74382e3c5d3218daa4363d by Nicolas Fella, on behalf
of Oded Arbel.
Committed on 20/12/2022 at 21:59.
Pushed by nicolasfella into branch 'master'.
Reduce use of KWindowSystem on Wayland
Related: bug 381541
This patchset was originally part of MR !81.
Problem description: Currently most of KWindowSystem standard API does not work
on Wayland, on purpose - the code for Wayland support was actually removed from
the KWindowSystem framework into a standalone
[plasma/kwayland-integration](https://invent.kde.org/plasma/kwayland-integration)
where
[src/windowsystem/windowsystem.cpp](https://invent.kde.org/plasma/kwayland-integration/-/blob/master/src/windowsystem/windowsystem.cpp)
has almost all API implementations empty and just issue warnings in the form of
"This plugin does not support...".
Additionally, in [bug report 442662: KWindowSystem on Wayland is missing many
signals, some of which are needed by
Yakuake](https://bugs.kde.org/show_bug.cgi?id=442662#c1) David Edmundson says
"Wayland is different, trying to shoehorn that into the exact same abstractions
simply won't work. Lets split this up as individual bugs on yakuake".
So this is some of that work of changing Yakuake Wayland behavior to not rely
on non-working (or mostly non-working) KWindowSystem and replace with
operations using Wayland APIs directly, or Qt API that is known to work with
Wayland.
This patch set is not complete in that:
a. There are some operations that are as of yet not possible under Wayland (for
example - set visible on all desktops), so removing those code calls under
Wayland will hide warnings that we may want to keep to remind us there is
missing behavior.
b. There are still some wayland-integration warnings that are shown when
running Yakuake on kwin_wayland. I did not work on these because changes do not
affect my use case (also see (a) above). If there is need, I can review more
APIs and do more KWindowSystem reduction.
M +17 -7 app/mainwindow.cpp
https://invent.kde.org/utilities/yakuake/commit/f0a2c72a6c52a16a0f74382e3c5d3218daa4363d
--
You are receiving this mail because:
You are watching all bug changes.
