Bug 488813

Summary: Shake cursor inoperable with animation speed set to instant
Product: [Plasma] kwin Reporter: Parker Reed <parker.l.reed>
Component: effects-variousAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: nate
Priority: NOR    
Version: 6.1.0   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/kwin/-/commit/64f1455b1153214e9825953609bd3b63bfd9c74a Version Fixed In: 6.1.1
Sentry Crash Report:

Description Parker Reed 2024-06-20 18:29:36 UTC 
***
If you're not sure this is actually a bug, instead post about it at https://discuss.kde.org

If you're reporting a crash, attach a backtrace with debug symbols; see https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***

SUMMARY

With animation speed set to instant, shake cursor does not work.

STEPS TO REPRODUCE
1. Set animations to instant
2. Enable Shake Cursor under accessibility 
3. Try to shake cursor

OBSERVED RESULT

Cursor stays the same size

EXPECTED RESULT

Cursor to inflate when shaken

SOFTWARE/OS VERSIONS 
Linux/KDE Plasma: Arch Linux
KDE Plasma Version: 6.1.0
KDE Frameworks Version: 6.3.0
Qt Version: 6.8.0
Comment 1 Parker Reed 2024-06-20 18:31:34 UTC 
Subsequently the cursor size changing is entirely dependent on animation speed.

If you have it set really low, it takes a long time to inflate and deflate.
Comment 2 Nate Graham 2024-06-20 19:09:22 UTC 
hah, confirmed. It's also pretty weird with the speed set to a very slow value.

Maybe we should opt this out of respecting the global animation scale.
Comment 3 Bug Janitor Service 2024-06-20 19:12:51 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/5951
Comment 4 Vlad Zahorodnii 2024-06-20 19:38:24 UTC 
Git commit 9276d28b3d6b5b8e572fdd46658734de54ef917f by Vlad Zahorodnii.
Committed on 20/06/2024 at 19:28.
Pushed by vladz into branch 'master'.

plugins/shakecursor: Ignore animation speed

From the user perspective, shaking cursor is not perceived as an
animation, so let's ignore the global animation speed factor.

M  +2    -4    src/plugins/shakecursor/shakecursor.cpp

https://invent.kde.org/plasma/kwin/-/commit/9276d28b3d6b5b8e572fdd46658734de54ef917f
Comment 5 Vlad Zahorodnii 2024-06-20 19:55:08 UTC 
Git commit 64f1455b1153214e9825953609bd3b63bfd9c74a by Vlad Zahorodnii.
Committed on 20/06/2024 at 19:41.
Pushed by vladz into branch 'Plasma/6.1'.

plugins/shakecursor: Ignore animation speed

From the user perspective, shaking cursor is not perceived as an
animation, so let's ignore the global animation speed factor.


(cherry picked from commit 9276d28b3d6b5b8e572fdd46658734de54ef917f)

Co-authored-by: Vlad Zahorodnii <vlad.zahorodnii@kde.org>

M  +2    -4    src/plugins/shakecursor/shakecursor.cpp

https://invent.kde.org/plasma/kwin/-/commit/64f1455b1153214e9825953609bd3b63bfd9c74a