Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Grey-out auto upload options until enabled #1260

Closed
mario opened this issue Aug 3, 2017 · 2 comments
Closed

Grey-out auto upload options until enabled #1260

mario opened this issue Aug 3, 2017 · 2 comments
Assignees
@mario
Labels
approved enhancement pr exists
Milestone
Nextcloud App 3.1.0

Comments

@mario
Copy link
Contributor

mario commented Aug 3, 2017 •
edited by AndyScherzinger
Loading

As suggested by @gahujipo here: #797 (comment)

The preferences window for auto upload shows a switch next to "Preferences for auto uploading". At the first sight it was not clear to me that that switch is uses to enable the auto upload feature for that folder pair. My idea was to grey out the options in the window unless the switch is set to true.
@mario mario added this to the Nextcloud App 1.5.1 milestone Aug 3, 2017
@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Aug 4, 2017 •
edited
Loading

Nice idea 👍
(silently assuming that you also like the idea as you have created this issue ;-) )

@mario
Copy link
Contributor Author

mario commented Aug 4, 2017

Yup, like the idea. That's called good UX :P

@AndyScherzinger AndyScherzinger modified the milestones: Nextcloud App 2.0.1, Nextcloud App 2.1.0 Oct 4, 2017
@AndyScherzinger AndyScherzinger modified the milestones: Nextcloud App 2.1.0, Nextcloud App 2.2.0 Nov 9, 2017
@mario mario self-assigned this Jan 28, 2018
mario added a commit that referenced this issue Feb 13, 2018
@mario
Fix #1260
5e222a1 
Signed-off-by: Mario Danic <mario@lovelyhq.com>
@mario mario mentioned this issue Feb 13, 2018
Merged
tobiasKaminsky pushed a commit that referenced this issue Apr 10, 2018
@mario @tobiasKaminsky
Fix #1260
dad3d2b 
Signed-off-by: Mario Danic <mario@lovelyhq.com>
tobiasKaminsky pushed a commit that referenced this issue Apr 10, 2018
@mario @tobiasKaminsky
Fix #1260
1f02e04 
Signed-off-by: Mario Danic <mario@lovelyhq.com>
AndyScherzinger added a commit that referenced this issue Apr 10, 2018
@AndyScherzinger
Merge pull request #2451 from nextcloud/backport_2164
09aa6d8 
Backport "Fix #1260"
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mario mario

Labels
approved enhancement pr exists
Projects
None yet
Milestone
Nextcloud App 3.1.0
Development

No branches or pull requests
3 participants
@mario @AndyScherzinger @tobiasKaminsky