hovancik/stretchly

Feature request: Disable break sub-options if the break is disabled

Ouverte

#773 ouverte le 18 nov. 2020

 (5 commentaires) (1 réaction) (2 personnes assignées)JavaScript (552 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (6 480 étoiles)
Métriques de merge PR
 (Merge moyen 2j 7h) (14 PRs mergées en 30 j)

Description

Prerequisites

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

  1. Toggle 'Show notification beforee Mini Break starts' to 'On'
  2. Toggle 'Enable Mini Breaks' to 'Off' image

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. image

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.

Guide contributeur