| Summary: | Screenshot taken on Wayland via keyboard shortcuts is not copied to clipboard | ||
|---|---|---|---|
| Product: | [Applications] Spectacle | Reporter: | Patrick Silva <bugseforuns> |
| Component: | General | Assignee: | Boudhayan Gupta <me> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | accounts, agurenko, bart, bugs-kde, cg.almehag, cristobal.veas, david.vuckovic7, iosifsi2263, kde.lwzr1, kde, kdebugs, kirill.bogdanenko, madLyfe, meven.car, meven29, nate, postix, qydwhotmail, supgesu, u.slash.vlad |
| Priority: | NOR | Keywords: | wayland |
| Version: | 21.08.2 | ||
| Target Milestone: | --- | ||
| Platform: | Manjaro | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/graphics/spectacle/commit/877744dfb824785e798ee7f322c4cc8b208a8aea | Version Fixed In: | 22.04 |
| Sentry Crash Report: | |||
|
Description
Patrick Silva
2020-11-20 11:34:24 UTC
*** This bug has been marked as a duplicate of bug 422860 *** Well this seemed to be the same 422860, but you are testing with Qt 5.15.2 where we should have the fix for 422860 I experience the problem with these specs: Operating System: Arch Linux KDE Plasma Version: 5.22.4 KDE Frameworks Version: 5.85.0 Qt Version: 5.15.2 Kernel Version: 5.13.13-arch1-1 (64-bit) Graphics Platform: Wayland I'm also having this problem on Manjaro's KDE release, fully updated: local/plasma-wayland-session 5.22.5-1 local/plasma-desktop 5.22.5-1 (plasma) local/plasma-framework 5.85.0-2 (kf5) local/qt5-base 5.15.2+kde+r215-1 (qt qt5) local/linux514 5.14.0-0 Can confirm to still be getting this on 5.23 Wayland: Operating System: openSUSE Tumbleweed 20211016 KDE Plasma Version: 5.23.0 KDE Frameworks Version: 5.86.0 Qt Version: 5.15.2 Kernel Version: 5.14.11-1-default (64-bit) Graphics Platform: Wayland Processors: 16 × AMD Ryzen 7 1800X Eight-Core Processor Memory: 15.6 GiB of RAM Graphics Processor: Radeon RX Vega I'm still affected by this bug. FYI we did the work for Qt6: https://codereview.qt-project.org/c/qt/qtwayland/+/360355 https://codereview.qt-project.org/c/qt/qtwayland/+/367306 And are now backporting things to KDE Qt Patch collection: First patch https://invent.kde.org/qt/qt/qtwayland/-/merge_requests/18 Then https://codereview.qt-project.org/c/qt/qtwayland/+/360355 will need to be backported too. This will be fixed then after the next arch-linux build of the KDE Qt Patch collection. Unfortunately not all distros uses our KDE Qt Patch collection. *** Bug 444038 has been marked as a duplicate of this bug. *** Well the deep down issue is Wayland disallow apps in the background to put things in the clipboard. The solution for this, is to create a dummy surface to be able to send data to the clipboard like the popup_surface in: https://github.com/bugaevc/wl-clipboard/blob/master/src/wl-copy.c *** Bug 445097 has been marked as a duplicate of this bug. *** *** Bug 444482 has been marked as a duplicate of this bug. *** A possibly relevant merge request was started @ https://invent.kde.org/graphics/spectacle/-/merge_requests/97 A possibly relevant merge request was started @ https://invent.kde.org/graphics/spectacle/-/merge_requests/97 Git commit 877744dfb824785e798ee7f322c4cc8b208a8aea by Méven Car, on behalf of Méven Car. Committed on 13/12/2021 at 10:03. Pushed by meven into branch 'master'. Wayland: Allow to copy to clipboard img while in dbus or background mode M +2 -1 CMakeLists.txt M +1 -0 src/CMakeLists.txt M +11 -24 src/ExportManager.cpp M +1 -1 tests/CMakeLists.txt https://invent.kde.org/graphics/spectacle/commit/877744dfb824785e798ee7f322c4cc8b208a8aea Do you think this is worth backporting, Méven? (In reply to Nate Graham from comment #15) > Do you think this is worth backporting, Méven? This is not simply feasible. This depends on KF 5.89 that just came out, while KDE Gears is already in dependency freeze. https://community.kde.org/Schedules/KDE_Gear_21.12_Schedule It could be adapted somewhat with #ifdef based on KF version, but expect from rolling distro that should not reach many users. Ah right, got it. (In reply to Méven Car from comment #16) > It could be adapted somewhat with #ifdef based on KF version, but expect > from rolling distro that should not reach many users. This is not true. The upcoming Ubuntu 22.04 LTS will have KDE Gears 21.12 in repos and already offers KF 5.89 (will most likely be updated again before release, feature freeze is in february). Ubuntu is a very well known distro, not rolling release and since the next release is a LTS one, this is what many user over many years will get. Please reconsider backporting in some form. Didn't know this bug was already reported (and fixed!) until I've read Nate's blog post. Great to see it finally fixed. Was bothering me ever since I've started using Wayland. This feature is just to good to use other ways :) Backport would be awesome to have! 22.04 seems like it's going to be light years away ;) (In reply to Elias from comment #18) > (In reply to Méven Car from comment #16) > > It could be adapted somewhat with #ifdef based on KF version, but expect > > from rolling distro that should not reach many users. > > This is not true. The upcoming Ubuntu 22.04 LTS will have KDE Gears 21.12 in > repos and already offers KF 5.89 (will most likely be updated again before > release, feature freeze is in february). > Ubuntu is a very well known distro, not rolling release and since the next > release is a LTS one, this is what many user over many years will get. > Please reconsider backporting in some form. This patch needs latest KDE Qt patch collection to work (https://community.kde.org/Qt5PatchCollection), ubuntu does only ships mainline Qt. Wayland support is simply very degraded without our patch collection. This is a downstream/packager issue. *** Bug 433681 has been marked as a duplicate of this bug. *** |