Page MenuHomePhabricator

Implement a confirmation dialogue when closing the app
Closed, ResolvedPublic

Assigned To
Authored By
Jpita
Sep 8 2020, 8:53 AM
Referenced Files
F33940435: 20201209_111649.jpg
Dec 9 2020, 5:51 AM
F33920586: image.png
Nov 20 2020, 4:51 PM
F33918206: image.png
Nov 18 2020, 10:08 AM
F33917260: image.png
Nov 17 2020, 10:06 AM
F32373921: Image from iOS.jpg
Oct 5 2020, 8:28 AM

Description

Issue

When a user uses the back key repeatedly, the app quits, but this behaviour is unexpected.

Request

Show a confirmation dialogue before closing the app for a better user experience

Copy/Message

Exit Wikipedia?
Are you sure you want to close the app?
LSK: Cancel
RSK: Exit

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

fyi, this is what in facebook app

Image from iOS.jpg (4×3 px, 1 MB)

AMuigai renamed this task from implement a confirmation dialogue when closing the kaios app to Implement a confirmation dialogue when closing the app.Nov 9 2020, 2:27 PM
AMuigai triaged this task as Medium priority.
AMuigai updated the task description. (Show Details)

Note: It requires update to our current confirm dialog UI

Current *1New
image.png (432×357 px, 18 KB)
image.png (320×240 px, 14 KB)

To be Confirmed :

*1 image here is another confirm dialog when navigate to the section within the same article, it has the CSK as the "confirm", but the new exit app confirm dialog has the RSK as the "confirm", do you want to make it a general confirm dialog or exception here?

@hueitan I would advise keeping the confirmation to RSK to avoid the accidental press and cancel to LSK. I also got a couple of questions for you.

  • Changing the dialog box's style would change it to other places(app language, article preview, etc.) as well?
  • Would it resize itself to accommodate translated copy?

@hueitan I would advise keeping the confirmation to RSK to avoid the accidental press and cancel to LSK. I also got a couple of questions for you.

OK, change to RSK

  • Changing the dialog box's style would change it to other places(app language, article preview, etc.) as well?

No, the confirm dialog exist only in two places (including one here)

  • Would it resize itself to accommodate translated copy?

Yes, It would auto extend the height of the message box

@hueitan I would advise keeping the confirmation to RSK to avoid the accidental press and cancel to LSK. I also got a couple of questions for you.

OK, change to RSK

  • Changing the dialog box's style would change it to other places(app language, article preview, etc.) as well?

No, the confirm dialog exist only in two places (including one here)

Ohk, got it.

  • Would it resize itself to accommodate translated copy?

Yes, It would auto extend the height of the message box

Great

image.png (662×534 px, 91 KB)

I think the language for the softkeys here is a bit confusing: we ask "Are you sure you want to close the app?" and then we use "Close" in the LSK, but the latter doesn't exit the app and instead closes the message box. I think "Yes" (RSK) or "No" (LSK) wold be less confusing (we have softkey-yes and softkey-no already in qqq.json)

fyi, this is what in facebook app

Or "Cancel" (LSK) and "Exit" (RSK) like in the facebook example, that could work too.

Thoughts?

@eamedina @hueitan let's use "Cancel" (LSK) and "Exit" RSK.

@hueitan has this been addressed in the PR?

@hueitan is it possible to center the "Exit Wikipedia" text in the header area? On the banana phone device, the tail of "p" is touching the main text area. Also, can we increase the height of the dialog box to a couple of pixels? as of now, it looks a bit busy.

20201209_111649.jpg (4×1 px, 347 KB)

@hueitan is it possible to center the "Exit Wikipedia" text in the header area? On the banana phone device, the tail of "p" is touching the main text area. Also, can we increase the height of the dialog box to a couple of pixels? as of now, it looks a bit busy.

20201209_111649.jpg (4×1 px, 347 KB)

why do we use "close" and "exit"?
shouldn't we use the same word?

@hueitan is it possible to center the "Exit Wikipedia" text in the header area? On the banana phone device, the tail of "p" is touching the main text area.

Yes! I saw that now!

Also, can we increase the height of the dialog box to a couple of pixels? as of now, it looks a bit busy.

The height of the dialog box is based on the content size, we can adjust the margin/padding of the content. and of course we can make the exception here, but what's UI in your mind right now?

Yeah, I agree with @Jpita.

@hueitan are you going to change the copy?
should I move it back to dev?

Make sense @Jpita, @hueitan how about we make this to "Are you sure you want to exit the app? Also, I am happy to jump on a call to tweak the margin/padding of the dialog box to see how it looks?

moving this to dev since it seems there will be some work (change a string) to be done