Skip to content
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

Don't close Export Dialog when the user clicks outside of it #557

Closed
sssoleileraaa opened this issue Sep 23, 2019 · 8 comments · Fixed by #666
Closed

Don't close Export Dialog when the user clicks outside of it #557

sssoleileraaa opened this issue Sep 23, 2019 · 8 comments · Fixed by #666

Comments

@sssoleileraaa
Copy link
Contributor

sssoleileraaa commented Sep 23, 2019

Description

Update:
#557 (comment)

Ensure clicking outside of the overlay prompts the user with an "Are you sure?" message when a transfer is in progress. This needs further discussion since we don't have a mechanism in place for canceling an export.

!https://user-images.githubusercontent.com/4522213/65459966-cacac580-de05-11e9-9771-bf8f50e5f9bd.png

@eloquence
Copy link
Member

I think we have to distinguish clearly between:

  1. An Are you sure you want to cancel? confirmation once the export process has started (we're past the pre-flight checks)
  2. An Are you sure you want to cancel? confirmation in earlier stages of the process.

My sense is that 1) is almost certainly off the table, given how difficult it is to observe or control the behavior of the export VM once the process has been kicked off. Instead, it may make more sense to prevent the overlay from being dismissed during this stage.

As for 2), IMO it could make sense to prompt the user before closing the overlay if and only if they have entered some text or performed other steps they would otherwise have to repeat. So, if you're in the middle of typing a long passphrase, clicking outside the overlay should probably bring up the confirmation dialog.

@sssoleileraaa
Copy link
Contributor Author

sssoleileraaa commented Oct 17, 2019

Another idea:

Don't allow the Export Dialog to close when the user clicks outside of it at any point during the export flow.

Instead we should ONLY provide a "Cancel" button.

That means when the user is prompted to insert their USB device or to enter their passphrase and continue, they should also have a cancel option (and that's it for canceling). Once the transfer begins, we can either a) keep the dialog open with an "Exporting" message until the file finishes transferring, or b) automatically close the dialog and display an async message to the user in the activity bar when the export finishes.

@sssoleileraaa
Copy link
Contributor Author

To visualize this idea, run:
--gpg-sign when making a commit and you'll see this behavior. You'll also see how it blocks user interaction with the Terminal and other applications when prompting the user to do something. Here's the "happy path" behavior:

  1. user is prompted to insert their yubikey with an option to continue or cancel
  2. user inserts yubikey and clicks continue
  3. user is prompted to enter their pin with a continue or cancel option
  4. user enters pin and clicks continue
  5. yubikey dialog immediately closes and you see yubikey flash a few times
  6. user can type in the terminal and do other things before the signing finishes, but eventually (very quickly in this case) the signing finishes and the terminal shows output showing a success.

@eloquence
Copy link
Member

eloquence commented Oct 21, 2019

Don't allow the Export Dialog to close when the user clicks outside of it at any point during the export flow.

We discussed this today, and agreed that this is the approach we want to take. @ninavizz will propose updated designs & workflows consistent with that approach.

Once the transfer begins, we can either a) keep the dialog open with an "Exporting" message until the file finishes transferring,

We've agreed to follow this approach for now. We can consider alternatives post-beta.

@ninavizz
Copy link
Member

Nit: "Design" is never a plural. "Design Solutions" is the appropriate plural.

Yes, many—too many—in tech, use the plural of "designs." Happy to engage in further chinrubbing if desired... but this'y here practitioner begs cessation of the plural of "designs."

P.S.: Yes, some designers also use this plural... but typically only the HCI-trained designers from academia, and not practitioners with actual design training. Design is a discipline, and its outputs are solutions or solution opportunities; its outputs are never "designs."

@eloquence
Copy link
Member

"design solutions" it is :P

@sssoleileraaa sssoleileraaa changed the title Canceling an export Don't close Export Dialog when the user clicks outside of it Nov 12, 2019
@sssoleileraaa
Copy link
Contributor Author

I nominate this for next sprint now that we've agreed to make this a modal instead of an overlay

@sssoleileraaa
Copy link
Contributor Author

This is a WIP, but it clearly shows that we will now show a "Done" (what I was referring to as "Continue" in previous comments) and "Cancel" option for all steps that you can cancel out of. The prep step and export-in-progress step will not have "Cancel" or "Done" options because you can't actually cancel them, but you can still "Close" the window.

Export_MessagingDraft_Nov19-01b.pdf

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

Successfully merging a pull request may close this issue.

3 participants