Bug 490703

Summary: Logout effect does not update animation duration in real-time
Product: [Plasma] kwin Reporter: Nate Graham <nate>
Component: effects-variousAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: 4wy78uwh
Priority: NOR    
Version: 6.1.3   
Target Milestone: ---   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=490531
Latest Commit: https://invent.kde.org/plasma/kwin/-/commit/b0a5918db9e281e8b680317eb35a7d20084e10c6 Version Fixed In: 6.2.0
Sentry Crash Report:

Description Nate Graham 2024-07-23 13:46:42 UTC 
STEPS TO REPRODUCE
1. Open System Settings
2. Change animation duration to anything other than its current value
3. Open Kickoff and click Shut Down, Restart, or Log Out

OBSERVED RESULT
The logout screen fades out with the old duration.

EXPECTED RESULT
The logout screen animates with the new duration.

ADDITIONAL INFORMATION
Restarting KWin or the computer makes it pick up the new duration. Same issue as Bug 490531 affecting the Show Desktop effect.
Comment 1 Bug Janitor Service 2024-07-24 13:39:44 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/6167
Comment 2 David Edmundson 2024-07-24 15:09:54 UTC 
Git commit b0a5918db9e281e8b680317eb35a7d20084e10c6 by David Edmundson.
Committed on 24/07/2024 at 15:09.
Pushed by davidedmundson into branch 'master'.

effects/logout Avoid caching animation duration

This is problematic as then we do not catch changes to animation speed
settings, it also doesn't make the code more readable when it's
only used in one place.

M  +2    -9    src/plugins/logout/package/contents/code/main.js

https://invent.kde.org/plasma/kwin/-/commit/b0a5918db9e281e8b680317eb35a7d20084e10c6