| Summary: | Windows opened in multiple desktops have wrong z-positioning when switching between desktops | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Komorebi <markovs.i.mail> |
| Component: | effects-various | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ericedlund2017, nate |
| Priority: | NOR | Keywords: | regression |
| Version First Reported In: | 5.25.5 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/commit/5ccb8f8d20933d89065708b83f9e62ff01d2e50a | Version Fixed/Implemented In: | 5.26 |
| Sentry Crash Report: | |||
|
Description
Komorebi
2022-08-04 15:23:08 UTC
Operating System: KDE neon 5.25 KDE Plasma Version: 5.25.4 KDE Frameworks Version: 5.96.0 Qt Version: 5.15.5 Kernel Version: 5.15.0-43-generic (64-bit) Graphics Platform: Wayland Processors: 16 × AMD Ryzen 7 5800HS with Radeon Graphics Memory: 22,9 GiB of RAM Graphics Processor: AMD RENOIR Manufacturer: ASUSTeK COMPUTER INC. Product Name: ROG Zephyrus G14 GA401QE_GA401QE System Version: 1.0 Can reproduce with the swipe effect; the issue is already fixed in current git master for Present Windows and Desktop Grid. But for the Swipe effect, the windows that are pinned to all desktops get raised to the top during the swipe. Then they go back to their correct Z order after the swipe is finished. It appears that windows on all desktops are being painted multiple times. There are two possible fixes: 1) Draw the window on each desktop and make it move with the slide 2) Draw the window stationary and make it look like it's moving between desktops This same bug effects notifications, should those be treated differently? Notifications are always supposed to appear above all windows, so I don't think it's a bug there. But we should fix windows. A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/2787 >the issue is already fixed in current git master for Present Windows and Desktop Grid
still present in 5.25.5
Git commit 6d9f740d6f02c7d8b8005edc3573c76b8f8f3cc3 by Vlad Zahorodnii, on behalf of Eric Edlund. Committed on 19/09/2022 at 07:00. Pushed by vladz into branch 'master'. Better handle windows on all desktops in slide effect Change the render cycle for slide effect. Now, windows will be drawn on each desktop or not in the paintWindow() loop instead of the paintScreen() loop. This allows us to respect stacking order better by painting all of the same window together instead of per VD. Related: bug 458622 M +75 -86 src/effects/slide/slide.cpp M +4 -8 src/effects/slide/slide.h https://invent.kde.org/plasma/kwin/commit/6d9f740d6f02c7d8b8005edc3573c76b8f8f3cc3 Git commit 5ccb8f8d20933d89065708b83f9e62ff01d2e50a by Vlad Zahorodnii, on behalf of Eric Edlund. Committed on 19/09/2022 at 07:03. Pushed by vladz into branch 'Plasma/5.26'. Better handle windows on all desktops in slide effect Change the render cycle for slide effect. Now, windows will be drawn on each desktop or not in the paintWindow() loop instead of the paintScreen() loop. This allows us to respect stacking order better by painting all of the same window together instead of per VD. Related: bug 458622 (cherry picked from commit 6d9f740d6f02c7d8b8005edc3573c76b8f8f3cc3) M +75 -86 src/effects/slide/slide.cpp M +4 -8 src/effects/slide/slide.h https://invent.kde.org/plasma/kwin/commit/5ccb8f8d20933d89065708b83f9e62ff01d2e50a |