| Summary: | Shake cursor makes it disappear instead of growing | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Julien Delquié <julien.dlq> |
| Component: | effects-various | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | agurenko, karl, kde.tarmac868, kdebugger, nate, nepnep, postix, vlad.zahorodnii |
| Priority: | VHI | Keywords: | regression |
| Version: | 6.1.1 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/-/commit/5d94ce9eac18a0ecf4dbbc8dbd04b385dc24c8ff | Version Fixed In: | KWin 6.1.1.1 |
| Sentry Crash Report: | |||
| Attachments: |
Changing cursors
qdbus org.kde.KWin kdedefaults/kcminputrc kcminputrc qdbus org.kde.KWin /KWin supportInformation The correct kcminputrc file patch debug patch updated patch |
||
|
Description
Julien Delquié
2024-06-26 14:31:18 UTC
I can confirm this on Fedora 40 and 6.1.1 However, if I change the cursor theme to something other than Breeze it works. Operating System: Fedora Linux 40 KDE Plasma Version: 6.1.1 KDE Frameworks Version: 6.3.0 Qt Version: 6.7.1 Kernel Version: 6.9.6-200.fc40.x86_64 (64-bit) Graphics Platform: Wayland So you're saying it disappears when you're using Breeze, but works fine with a 3rd-party cursor theme? I can't reproduce the issue, and interestingly enough it's the opposite of what I saw in https://invent.kde.org/plasma/kwin/-/merge_requests/5967 while reviewing it. It got fixed before merge though. (In reply to Nate Graham from comment #2) > So you're saying it disappears when you're using Breeze, but works fine with > a 3rd-party cursor theme? > > I can't reproduce the issue, and interestingly enough it's the opposite of > what I saw in https://invent.kde.org/plasma/kwin/-/merge_requests/5967 while > reviewing it. It got fixed before merge though. Yes exactly right. I've tried numerous 3rd party cursor themes and it works fine. Created attachment 171019 [details]
Changing cursors
A brief screen cast showing the issue.
Can somebody change the cursor theme to breeze and attach ~/.config/kdedefaults/kcminputrc and ~/.config/kcminputrc to this bug report? Also can you post the output of `qdbus org.kde.KWin /KWin supportInformation`? How did you get 6.1.1 in Fedora 40, Colin? They're not shipping it to stable yet. Did you use a COPR or something? . (In reply to Nate Graham from comment #6) > How did you get 6.1.1 in Fedora 40, Colin? They're not shipping it to stable > yet. Did you use a COPR or something? I use Solopasha's COPR Nate, that way I can test beta's/early releases :) Created attachment 171031 [details]
qdbus org.kde.KWin
Created attachment 171032 [details]
kdedefaults/kcminputrc
Created attachment 171033 [details]
kcminputrc
(In reply to Vlad Zahorodnii from comment #5) > Can somebody change the cursor theme to breeze and attach > ~/.config/kdedefaults/kcminputrc and ~/.config/kcminputrc to this bug > report? Also can you post the output of `qdbus org.kde.KWin /KWin > supportInformation`? Done qdbus command output is wrong. Can you please run "qdbus org.kde.KWin /KWin supportInformation" ? Also do effects such as invert work correctly? (In reply to Vlad Zahorodnii from comment #13) > qdbus command output is wrong. > > Can you please run "qdbus org.kde.KWin /KWin supportInformation" OK > Also do effects such as invert work correctly? Seems OK, I will also attach the correct kcminput file Created attachment 171036 [details]
qdbus org.kde.KWin /KWin supportInformation
Created attachment 171037 [details]
The correct kcminputrc file
Configuration is fine. Maybe Qt resources (high resolution breeze cursor theme) have been stripped from kwin executables during build? Created attachment 171039 [details]
patch
@Julien can you please apply the following patch? then shake the cursor and post kwin output here
I have applied the patch and found something like this in journalctl:
juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: :/effects/shakecursor: QList("cursors")
juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: :/effects/shakecursor/cursors: QList("breeze_cursors", "Breeze_Light")
juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: :/effects/shakecursor/cursors/breeze_cursors: QList("cursors", "index.theme")
juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: :/effects/shakecursor/cursors/breeze_cursors/cursors: QList("default")
Is it what you wanted me to show you?
(In reply to Julien Delquié from comment #19) > I have applied the patch and found something like this in journalctl: > > juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: > :/effects/shakecursor: QList("cursors") > juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: > :/effects/shakecursor/cursors: QList("breeze_cursors", "Breeze_Light") > juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: > :/effects/shakecursor/cursors/breeze_cursors: QList("cursors", "index.theme") > juin 26 21:50:29 pasokon.maison kwin_wayland_wrapper[3374]: > :/effects/shakecursor/cursors/breeze_cursors/cursors: QList("default") > > Is it what you wanted me to show you? Yeah, it looks fine... I don't understand why the cursor is hidden then... I just ran a test, and downgraded kwin* to 6.1.0, then ran kwin --replace. All my cursors including Breeze worked fine. I then updated back to kwin 6.1.1 and the Breeze cursor disappears on shake. Just FYI Created attachment 171046 [details]
debug patch
@Julien can you please test another patch?
Created attachment 171047 [details]
updated patch
Another report from Neon: https://discuss.kde.org/t/mouse-cursor-disappears-when-shaken-and-then-reappears-after-a-few-seconds/17827 :( (In reply to Vlad Zahorodnii from comment #22) > Created attachment 171046 [details] > debug patch > > @Julien can you please test another patch? This patch does not apply: * Source directory (CMAKE_USE_DIR): "/var/tmp/portage/kde-plasma/kwin-6.1.1/work/kwin-6.1.1" * Build directory (BUILD_DIR): "/var/tmp/portage/kde-plasma/kwin-6.1.1/work/kwin-6.1.1_build" * ============================================================================================================================================================= * Applying user patches from /etc/portage/patches ... * Applying 489241-2.patch ... patching file src/utils/xcursortheme.cpp Hunk #1 FAILED at 102. Hunk #2 succeeded at 104 with fuzz 2 (offset -21 lines). Hunk #3 succeeded at 122 (offset -21 lines). Hunk #4 succeeded at 135 (offset -21 lines). Hunk #5 succeeded at 191 (offset -21 lines). 1 out of 5 hunks FAILED -- saving rejects to file src/utils/xcursortheme.cpp.rej [ !! ] (In reply to Vlad Zahorodnii from comment #23) > Created attachment 171047 [details] > updated patch nor this one: * Applying 489241-3.patch ... patching file src/utils/xcursortheme.cpp Hunk #2 FAILED at 103. Hunk #3 succeeded at 105 with fuzz 2 (offset -21 lines). Hunk #4 succeeded at 123 (offset -21 lines). Hunk #5 succeeded at 136 (offset -21 lines). Hunk #6 succeeded at 192 (offset -21 lines). I tried these patches (171046 and 171047) with or without first patch (171039), same results. A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/6002 Git commit 7527a9099cfc3f0127b87ddad488c8059cdf3a8d by Vlad Zahorodnii. Committed on 27/06/2024 at 07:32. Pushed by vladz into branch 'Plasma/6.1'. 3rdparty: Drop xcursor write hook It's unused. (cherry picked from commit d655b5a994bb62caf40a92c4c11c1903f386e441) M +0 -9 src/3rdparty/xcursor.c https://invent.kde.org/plasma/kwin/-/commit/7527a9099cfc3f0127b87ddad488c8059cdf3a8d Git commit 5d94ce9eac18a0ecf4dbbc8dbd04b385dc24c8ff by Vlad Zahorodnii. Committed on 27/06/2024 at 07:32. Pushed by vladz into branch 'Plasma/6.1'. utils: Load Xcursor themes using QFile This improves the integration of the XcursorTheme class with the rest of the Qt ecosystem, for example it should be possible to load cursor themes from Qt resources. (cherry picked from commit 586a775d5b65dc14458f6e861515f9f9b35f6cad) M +6 -54 src/3rdparty/xcursor.c M +10 -1 src/3rdparty/xcursor.h M +22 -1 src/utils/xcursortheme.cpp https://invent.kde.org/plasma/kwin/-/commit/5d94ce9eac18a0ecf4dbbc8dbd04b385dc24c8ff Git commit af9b3dd54e54806c435b8e6ed51cd3808d84f7a4 by Vlad Zahorodnii.
Committed on 27/06/2024 at 07:33.
Pushed by vladz into branch 'Plasma/6.1'.
3rdparty: Reformat xcursor.{h,c}
(cherry picked from commit e678ebef2983dff97f6ee744d9b2f9770e1f21d0)
M +224 -242 src/3rdparty/xcursor.c
M +26 -24 src/3rdparty/xcursor.h
https://invent.kde.org/plasma/kwin/-/commit/af9b3dd54e54806c435b8e6ed51cd3808d84f7a4
I have applied patches from merge request. It works! (And also I can see that cursor is really pretty now, when magnified) *** Bug 489300 has been marked as a duplicate of this bug. *** (In reply to Julien Delquié from comment #32) > I have applied patches from merge request. It works! (And also I can see > that cursor is really pretty now, when magnified) Thanks for confirming that the fix works! :) Thanks for the fast fix, confirmed working on this Fedora box as well. *** Bug 489323 has been marked as a duplicate of this bug. *** *** Bug 489388 has been marked as a duplicate of this bug. *** |