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

Dialogs with multiple possible actions look bad #1265

Closed
lognaturel opened this issue Jul 20, 2017 · 20 comments
Closed

Dialogs with multiple possible actions look bad #1265

lognaturel opened this issue Jul 20, 2017 · 20 comments

Comments

@lognaturel
Copy link
Member

lognaturel commented Jul 20, 2017

It's no longer very clear that save or ignore changes are clickable:

I looked through the material design guidelines and would like to propose something like the following:
screen shot 2017-07-19 at 5 32 59 pm
Disclaimer, I am not a designer and certainly open to other ideas! The icons should probably be tinted with the accent color. @shobhitagarwal1612 it would be good to get your feedback.

From what I understand, material design would suggest automatically saving and not presenting this kind of dialog at all. This can certainly be reviewed at some point, but for now I think it would be best to make the dialog more usable and fit in with other dialogs that now roughly follow material design guidelines.

Note that the guidelines explicitly state that this kind of dialog should have no cancel button and that the user should just use the back button or tapping outside the dialog to dismiss it. The cancel button has been on that dialog for a long time, though, and I think it's better not to assume that users know those other ways of dismissing dialogs.

@shobhitagarwal1612
Copy link
Contributor

shobhitagarwal1612 commented Jul 25, 2017

Sorry for the late response. How about breaking this into two dialogs?

First one would be :

Exit All Dialogs?

        No   Yes 

If user presses Yes then :

Save changes?

      No   Yes

@YuraLaguta
Copy link
Contributor

IMHO two clicks will be more annoying.
When I'm exiting Writing a new email screen in (Gmail app) it's Saves it automatically as a draft. And shows Toast.

Calendar on the other hand shows:
screenshot_20170817-193751
when I try to exist just create event.

Should we drop the icons? It looks much cleaner without them.

Maybe something like:
"Leaving All Widget form with unsaved changes?"
'Cancel' 'Discard' 'Save'

@lognaturel
Copy link
Member Author

The double dialog does seem like it would be more disruptive so I'm not sure I like it very much.

@Yurii-Laguta when you say drop the icons, do you just mean leaving it as it is currently? Perhaps just making sure cancel is on the right and in uppercase and dropping the icon would make it look better. I wouldn't be opposed to doing just that.

The problem with three buttons side by side is that we do need to support small screens and multiple languages.

@YuraLaguta
Copy link
Contributor

@lognaturel I mean don't show any icons. On the second thought Action icons look good, but not the title.

I see the 3 button won't work.

How about Bottom sheet dialog? Similar to Google Photos app, it's nicely dismissible with swipe down or back button, will it be too much learning curve for user?
Another advantage it's close to the back button, so it's less disruptive than dialog in the centre.
screenshot_20170818-201557

@YuraLaguta
Copy link
Contributor

Started working on it.
[Off topic] is there a way to self assign issue?

@YuraLaguta
Copy link
Contributor

YuraLaguta commented Aug 19, 2017

@lognaturel is that something you had in mind?:
screenshot 2017-08-19 15 59 24
To me this one looks better:
screenshot 2017-08-19 16 31 56

@YuraLaguta
Copy link
Contributor

Here is version with bigger icons and smaller text
screenshot 2017-08-19 16 44 55

@lognaturel
Copy link
Member Author

@Yurii-Laguta unfortunately there isn't a way to self-assign issues yet but hopefully that will come very soon. See https://forum.opendatakit.org/t/fine-grained-permissions-for-github-issues-self-assign-labels/9352 for where we're at and please jump in if you have any other ideas!

I am intrigued by the bottom sheet idea and I hadn't noticed that interface before. I think the situation may be a little different. In the Google Photos, the bottom sheet shows up when a user taps the trashcan icon. The back button hides the bottom sheet when it's out. In the Collect case, the back button would trigger the bottom sheet. I suppose tapping the back button again would also hide the sheet / cancel. That seems like it could be a somewhat unexpected interaction.

@wonderchook, @hooverlunch you both have a good feel for what your respective users need. Do you have any opinions on this issue and proposed changes?

@smoyte
Copy link

smoyte commented Aug 21, 2017

This all looks good to me. :)

@YuraLaguta
Copy link
Contributor

@lognaturel very good point, I see now that bottom sheet is not suitable for this use case.
Please let me know if any changes required.

@lognaturel
Copy link
Member Author

The form exit dialog has been updated. It would be good to reuse the same layout and look for the draw/signature/annotate widgets.

@shobhitagarwal1612
Copy link
Contributor

Fixed in #1360

@lognaturel
Copy link
Member Author

Let's keep open until the draw/signature/annotate menu uses the same look and feel!

@YuraLaguta
Copy link
Contributor

YuraLaguta commented Sep 3, 2017 via email

@lognaturel
Copy link
Member Author

Awesome, thanks @Yurii-Laguta! Take a look at createQuitDrawDialog() in DrawActivity.

@lognaturel lognaturel added in progress and removed bug labels Sep 3, 2017
@getodk-bot
Copy link
Member

ERROR: This issue is in progress, but has no assignee.

@YuraLaguta
Copy link
Contributor

YuraLaguta commented Sep 24, 2017

@opendatakit-bot assignToMe

@lognaturel
Copy link
Member Author

Our poor bot is not all that clever, @Yurii-Laguta. 😕 The only command it understands to assign issues is "claim" so you would type opendatakit-bot claim. You can check out the other commands it understands at https://github.com/opendatakit/zulipbot/blob/master/src/config.js

@YuraLaguta
Copy link
Contributor

thanks @lognaturel
@opendatakit-bot claim

@getodk-bot
Copy link
Member

Welcome to Open Data Kit, @Yurii-Laguta! We just sent you an invite to collaborate on this repository at https://github.com/opendatakit/collect/invitations. Please accept this invite in order to claim this issue and begin a fun and rewarding experience contributing to Open Data Kit!

Here are some tips to get you off to a good start:

  • Please read the README.md and CONTRIBUTING.md in this repo. Those two documents have much of what you need to get started.
  • Join the ODK developer Slack to get help, chat about this issue, and meet the other developers.
  • Sign up and introduce yourself on the ODK community forum to meet the broader ODK community.

See you on the other side (that is, the pull request side)!

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

No branches or pull requests

5 participants