Bug 475061

Summary: System Settings ignoring failsafe timeout if window is closed before time expires.
Product: [Applications] systemsettings Reporter: Wesley M <wmprivacyemail>
Component: kcm_kscreenAssignee: kscreen-bugs-null <kscreen-bugs-null>
Status: CONFIRMED ---    
Severity: minor CC: nate, plasma-bugs-null
Priority: NOR Keywords: usability
Version First Reported In: 5.27.8   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Video Example

Description Wesley M 2023-09-30 18:18:19 UTC
Created attachment 161975 [details]
Video Example

SUMMARY
***
System Settings ignoring failsafe timeout if window is closed before time expires.
***


STEPS TO REPRODUCE
1. Open display settings.
2. Adjust a setting that uses a timeout failsafe to revert changes if the applied changes break the system and the user is unable to interact with settings window within the timeout period.
3. Close the display settings window before the timeout period expires.

OBSERVED RESULT
When the window is closed, the changes appear to be permanently saved.

EXPECTED RESULT
The changes should be reverted, as the user did not approve the applied changes during the failsafe timeout period. If the window is accidentally closed, crashes, the desktop environment restarts, or the system is shut down; then it should revert to the last known-good settings, instead of potentially trapping the user in a misconfigured or unusable system.

SOFTWARE/OS VERSIONS
Windows: n/a
macOS: n/a
Linux/KDE Plasma: Fedora 38
(available in About System)
KDE Plasma Version: 5.27.8
KDE Frameworks Version: 5.109.0
Qt Version: 5.15.10

ADDITIONAL INFORMATION
Note that the video example shows a different issue with OBS recording the cursor position incorrectly after the resolution change. It is unrelated to this bug report. However, it can be used to show the resolution was changed and did not revert after the window closed.
Comment 1 Nate Graham 2023-10-11 19:30:53 UTC
At the moment I believe this is intended, but it could be changed.