hovancik/stretchly
View on GitHubFeature request: Disable break sub-options if the break is disabled
Open
#773 opened on Nov 18, 2020
good first issuehelp wanted
Repository metrics
- Stars
- (3,892 stars)
- PR merge metrics
- (Avg merge 2d 7h) (14 merged PRs in 30d)
Description
Prerequisites
- I'm using latest version: v
- I've checked Known issues
- I've checked Open issues
Description
I realised when I disabled Mini breaks, I would still get the notification before mini break starts. That's because I forgot to turn the option 'Show notification before Mini Break starts' option off, it might be a good idea to disable any sub-options as well (grey out) if the main break option is disabled.
Operating system: Windows 10
Steps to Reproduce
- Toggle 'Show notification beforee Mini Break starts' to 'On'
- Toggle 'Enable Mini Breaks' to 'Off'

Expected behavior:
- The sub-options for showing a notification or enable postponement are unchecked OR
- The sub option values are kept but they are greyed out and not active.
- May be even consider hiding the the Break for: and Every: options (if Mini breaks is not enabled then the sub options are probably just taking up extra space)
Actual behavior: [What actually happens]
The notification still came up for me.

Reproduces how often: [What percentage of the time does it reproduce?]
Can't check at the moment, but I can report back when I get a chance.