Bug 350828 - Diashow interval seconds cannot be set to zero
Summary: Diashow interval seconds cannot be set to zero
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Image & Slideshow wallpaper plugins (other bugs)
Version First Reported In: 5.3.2
Platform: Arch Linux Linux
: NOR minor
Target Milestone: 1.0
Assignee: Marco Martin
URL:
Keywords:
: 360970 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-07-31 17:08 UTC by jhnnslschnr
Modified: 2016-04-06 07:18 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.7.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description jhnnslschnr 2015-07-31 17:08:46 UTC
The spinbox range is correctly adjusted to start from 0 after settings minutes to something greater than 0, but it gets saved as 1 nevertheless (reopening the dialog shows this).

Reproducible: Always
Comment 1 David Edmundson 2015-08-02 22:53:52 UTC
Confirmed.
Comment 2 David Edmundson 2015-08-02 22:57:33 UTC
FYI, it can be set to 0. Just next time you open the config dialog it shows the wrong thing.
Comment 3 Kai Uwe Broulik 2015-08-07 12:13:34 UTC
Because it's told to do so:
The seconds spinbox minimum value is: minutesInterval.value == 0 && hoursInterval.value == 0 ? 1 : 0
Comment 4 David Edmundson 2015-08-07 12:16:39 UTC
aye, the idea being you can't have the time on complete 0, but can set it to 1 minute, 0 seconds.

However, due to declarative nature of things it means when we load the UI; everything is 0, minimum gets set to 1, which means we automatically change the value.
Comment 5 Kai Uwe Broulik 2015-08-07 12:46:30 UTC
Ah, I thought the bug was about not being able to set the total interval to zero. Sorry for the noise :)
Comment 6 Kai Uwe Broulik 2016-03-25 14:28:30 UTC
*** Bug 360970 has been marked as a duplicate of this bug. ***
Comment 7 Mircea Kitsune 2016-03-25 15:19:44 UTC
Confirmed. I'm getting this ever since I switched to Plasma 5. I would like to see it solved if possible, since it's a rather obvious aesthetical problem in the menu. I wrote a more in depth description in my own report, if it's of any help: https://bugs.kde.org/show_bug.cgi?id=360970
Comment 8 Kai Uwe Broulik 2016-04-06 07:18:18 UTC
Git commit 7656fa02e680bbc85dd03635149cc6643f5aa51a by Kai Uwe Broulik.
Committed on 06/04/2016 at 07:17.
Pushed by broulik into branch 'master'.

[Slideshow Wallpaper] Fix seconds always set to 1 when opening config dialog
FIXED-IN: 5.7.0

Differential Revision: https://phabricator.kde.org/D1221

M  +8    -17   wallpapers/image/imagepackage/contents/ui/config.qml

http://commits.kde.org/plasma-workspace/7656fa02e680bbc85dd03635149cc6643f5aa51a