-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Unable to exit the app from onbarding tutorial #3590
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
Comments
@sivaraam I think it's a minor change. will fix this. |
It's done, should i submit a PR for this? |
@318anushka Sure, feel free to submit a PR :) |
@sivaraam on beta we provide a Skip Tutorial button but on prod, it's intentionally hidden from there so we should be giving this feature only for beta IMO. |
@madhurgupta10 I don't get it. Why should we prevent the user from exiting the tutorial in any case? Wouldn't that be intrusive? It's his wish if he wants to exit. As mentioned in the description, we can show the tutorial again if he hasn't completed it. Also, as of now the user can exit by tapping on home or removing app from recent apps. We're just making sure they can exit via the back button too for ease of use. |
Yes, this is clearly a bug, the back button should take him to the previous activity or so exit the app if there are no activities in the stack |
Exactly. This is what I was thinking too. Jus in case it's not clear, I'm speaking of exiting the app by tapping on back from the first tutorial screen. I'm not speaking of exiting the app when the user is in the middle of the tutorial. |
We don't want to exit from app when user again go through the tutorial after login, right? |
Right.
Exactly. Also, I think there's at least one other case in which the tutorial is shown. When you have more number of deletions (> 50%) the app prompts you to take a quiz. If you answer that quiz badly it asks you go through the tutorial, AFAIR. |
I wanted to share that the prod do not have the Skip Tutorial button which was intentionally hidden (which I assume to force new users to go through the tutorial) but as you said it doesn't make much sense to prevent a user from skipping the tutorial. So we should allow the user to quit the tutorial by pressing the back button and also show the Skip Tutorial button for prod also. |
@madhurgupta10 Actually, it's not about skipping tutorial but exiting the app on pressing back button in the first screen of tutorial. |
Okay, now I see the bug. I was trying it by launching the Tutorial from the Navigation Drawer. |
Summary:
After a fresh install, we show the onboarding tutorial to the users to educate them about Wikimedia Commons and the kinds of pictures they can upload. That's nice. But it's impossible to exit the tutorial using the back button. This seems too restrictive and unreasonable, IMO. An user should be able to exit the tutorial without issues. We can show them the tutorial again upon next open if they haven't completed it.
Steps to reproduce:
Device and Android version:
Samsung SM-J111, Android 5.1.1
Commons app version:
2.12.3.629~a63a358
The text was updated successfully, but these errors were encountered: