Unable to exit the app from onbarding tutorial

[sivaraam]

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]

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

[318anushka]

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

[sivaraam]

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

[madhurgupta10]

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

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

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]

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]

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]

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.