Bug 490703 - Logout effect does not update animation duration in real-time
Summary: Logout effect does not update animation duration in real-time
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: effects-various (show other bugs)
Version: 6.1.3
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-07-23 13:46 UTC by Nate Graham
Modified: 2024-07-27 09:20 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 6.2.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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