Skip to content

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

Closed
sivaraam opened this issue Mar 28, 2020 · 12 comments · Fixed by #3597
Closed

Unable to exit the app from onbarding tutorial #3590

sivaraam opened this issue Mar 28, 2020 · 12 comments · Fixed by #3597
Assignees

Comments

@sivaraam
Copy link
Member

sivaraam commented Mar 28, 2020

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:

  1. Install the app
  2. Open the app
  3. Confirm that the tutorial screen is shown
  4. Tap on back to exit the app
  5. Observe that it's not possible to exit the app

Device and Android version:
Samsung SM-J111, Android 5.1.1

Commons app version:
2.12.3.629~a63a358

@318anushka
Copy link
Contributor

@sivaraam I think it's a minor change. will fix this.

@318anushka
Copy link
Contributor

It's done, should i submit a PR for this?

@sivaraam
Copy link
Member Author

@318anushka Sure, feel free to submit a PR :)

@madhurgupta10
Copy link
Collaborator

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

@sivaraam
Copy link
Member Author

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

@ashishkumar468
Copy link
Collaborator

ashishkumar468 commented Mar 31, 2020

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

@sivaraam
Copy link
Member Author

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.

@318anushka
Copy link
Contributor

We don't want to exit from app when user again go through the tutorial after login, right?
In that case we want to go to the previous activity.
Have to modify the code because in that case too, on pressing back app closes clearing the backstack.

@sivaraam
Copy link
Member Author

sivaraam commented Mar 31, 2020

We don't want to exit from app when user again go through the tutorial after login, right?

Right.

In that case we want to go to the previous activity.

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.

@madhurgupta10
Copy link
Collaborator

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

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.

Screenshot_20200331-224302
Screenshot_20200331-224244

@318anushka
Copy link
Contributor

318anushka commented Mar 31, 2020

@madhurgupta10 Actually, it's not about skipping tutorial but exiting the app on pressing back button in the first screen of tutorial.
Fresh install the app and open it and try to exit the app by pressing back button(keep pressing until you reach first screen). It won't, first screen of tutorial keeps popping up.

@madhurgupta10
Copy link
Collaborator

@madhurgupta10 Actually, it's not about skipping tutorial but exiting the app on pressing back button in the first screen of tutorial.
Fresh install the app and open it and try to exit the app by pressing back button(keep pressing until you reach first screen). It won't, first screen of tutorial keeps popping up.

Okay, now I see the bug. I was trying it by launching the Tutorial from the Navigation Drawer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants