-
Notifications
You must be signed in to change notification settings - Fork 43
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.
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
Feature Update: Warning on 'Accidental' Exit from Main Screen #202
Comments
Yes, I too think this will enhance the user experience. After clicking on the back button we can ask user to click back again to exit the app. |
@IAmJaishree Yeah Exactly! |
The issue is that Services is called from ODK-X Tables and ODK-X Survey. The 3 work together so it appears seamless to move between the applications. Therefore it will be akward for the user to have to confirm exit when they did not even realize they switched from Tables to Services (or Survey to Services). |
@wbrunette Yes I agree that would be awkward, but this feature would be implemented only on the MainActivity of Services Application and that too on Pressing the Back Button. Thus it would have no effect on the Current Flow between the 3 Applications. |
@VarunT11 I agree that only on MainActivity could solve the problem, as long as there are no specific inter-app functionality that depends on MainActivity. I can bring this up for discussion. I am slightly worried about the snackbar as that people will miss the message at the bottom and be frustrated. I think this needs a discussion with the PMC/TSC. I am happy to bring it up at the next meeting. You can start to implement it if you want, but changes may come from the committee. |
@wbrunette Regarding the Snackbar, we can decide a convenient time duration between 2 to 4 seconds of displaying the Snackbar such that a User gets sufficient time to read the warning message and then perform the next operation accordingly. |
Current Scenario:
Issue:
Proposed Solution:
Reference:
PS: If this Feature Update appears to be a valid one, I would like to work on it.
The text was updated successfully, but these errors were encountered: