| Summary: | After changing OSD time with keyboard and then up/down arrows of the spinbox, delay time resets to 0 ms on applying | ||
|---|---|---|---|
| Product: | [Applications] systemsettings | Reporter: | Patrick Silva <bugseforuns> |
| Component: | kcm_kwinvirtualdesktops | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate, plasma-bugs |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/commit/b8580339b44e376dbf035cb7a93096f71319ee54 | Version Fixed In: | 5.22 |
| Sentry Crash Report: | |||
| Attachments: | screeen recording | ||
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/1024 Git commit 5484b846b3869a8adca4cf801ca296b94017179e by Nate Graham, on behalf of David Edmundson. Committed on 24/05/2021 at 18:51. Pushed by ngraham into branch 'master'. [kcmkwin/kwindesktop] Implement valueFromText in a spinbox If textFromValue is set, valueFromText must be. A lot of other implemenetations were using parseInt() which fails at reading "3,000" due to the comma. Qt's fromLocaleString handles that but needs any text stripping. This implementation isn't perfect, but really we need to work on a good upstream solution. M +1 -0 src/kcmkwin/kwindesktop/package/contents/ui/main.qml https://invent.kde.org/plasma/kwin/commit/5484b846b3869a8adca4cf801ca296b94017179e Git commit b8580339b44e376dbf035cb7a93096f71319ee54 by Nate Graham, on behalf of David Edmundson. Committed on 24/05/2021 at 19:11. Pushed by ngraham into branch 'Plasma/5.22'. [kcmkwin/kwindesktop] Implement valueFromText in a spinbox If textFromValue is set, valueFromText must be. A lot of other implemenetations were using parseInt() which fails at reading "3,000" due to the comma. Qt's fromLocaleString handles that but needs any text stripping. This implementation isn't perfect, but really we need to work on a good upstream solution. (cherry picked from commit 5484b846b3869a8adca4cf801ca296b94017179e) M +1 -0 src/kcmkwin/kwindesktop/package/contents/ui/main.qml https://invent.kde.org/plasma/kwin/commit/b8580339b44e376dbf035cb7a93096f71319ee54 |
Created attachment 138662 [details] screeen recording SUMMARY Please watch the attached screen recording. EXPECTED RESULT never resets OSD time after click on Apply button SOFTWARE/OS VERSIONS Operating System: KDE neon Unstable Edition KDE Plasma Version: 5.22.80 KDE Frameworks Version: 5.83.0 Qt Version: 5.15.2 Graphics Platform: X11