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

server settings - confirm deletion #423

Closed
bceylon opened this issue Jan 6, 2017 · 3 comments
Closed

server settings - confirm deletion #423

bceylon opened this issue Jan 6, 2017 · 3 comments
Labels
enhancement

Comments

@bceylon
Copy link

bceylon commented Jan 6, 2017

Platform & OS Version
iOS 10

The version of the app you are reporting:
1.2.35

Device details:
iPhone SE & iPad

What is the nature of your issue
Event server settings should ask confirmation before deleting a server.

Details
Even though Delete is in red, it should still ask confirmation before actually deleting server settings. User can easily tap this by accident. Also, for the first time users it's not that obvious what is being deleted. You can easily think that clicking Delete would open some subsequent dialog or similar from which you'd select what to delete. Currently it just goes and deletes current settings without any confirmations.
@pliablepixels
Copy link
Member

pliablepixels commented Jan 6, 2017 •
edited
Loading

Deleted previous comment - I think you are only asking I add a confirmation box before delete - yes, I'll do that - agreed

@pliablepixels pliablepixels changed the title Event server settings - confirm deletion server settings - confirm deletion Jan 6, 2017
pliablepixels added a commit that referenced this issue Jan 6, 2017
@pliablepixels
confirm deletion of profiles #423
adb6a8c
@pliablepixels
Copy link
Member

done

pliablepixels added a commit that referenced this issue Jan 6, 2017
@pliablepixels
confirmation text #423
e91f7c2
pliablepixels added a commit that referenced this issue Jan 6, 2017
@pliablepixels
zone editing UI framework sort of done - no backend code, so hiding f…
7d019a5 
…or now #423
maymaymay added a commit to maymaymay/zmNinja that referenced this issue Jan 7, 2017
@maymaymay
Translation update to ZoneMinder#423
6a81adc
@pliablepixels
Copy link
Member

done

pliablepixels added a commit that referenced this issue Jan 16, 2017
@pliablepixels
Merge pull request #425 from maymaymay/patch-3
18521f6 
Translation update to #423
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pliablepixels @bceylon