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

[UX] Better module uninstallation confirm form. #3161

Closed
klonos opened this issue Jun 3, 2018 · 11 comments
Closed

[UX] Better module uninstallation confirm form. #3161

klonos opened this issue Jun 3, 2018 · 11 comments

Comments

@klonos
Copy link
Member

klonos commented Jun 3, 2018

Before:

screen shot 2018-07-31 at 2 40 20 pm

After:

screen shot 2018-07-31 at 2 34 18 pm


PR by @klonos: backdrop/backdrop#2220

@klonos
Copy link
Member Author

klonos commented Jun 3, 2018

In this PR:

  • moved the "Would you like ..." text to the confirmation page title (same as done for confirming deleting content)
  • Converted the "all data will be lost" text to a warning (we were previously emphasising it using <em> tags)
  • removed the now redundant "Would you like to continue with uninstalling the above?" question, as it was repeating what is already being asked in the form title.
  • changed "Would you like ...?" to "Are you sure ...?"

@olafgrabienski
Copy link

Are you sure you want to continue with uninstalling the modules listed bellow?

@klonos There's a small typo at the end of the sentence: "bellow" (two l)

@klonos
Copy link
Member Author

klonos commented Jun 4, 2018

Good catch @olafgrabienski 👍 ...fixed!

@opi
Copy link

opi commented Jun 4, 2018

In the 2nd screenshot, the lonely bullet list without heading feels a little lost

@klonos
Copy link
Member Author

klonos commented Jun 4, 2018

I hear ya @opi, but the text that was previously there has been moved to the page title and the warning. There was nothing left 😄 ...any suggestion?

@quicksketch
Copy link
Member

I often find that putting an entire sentence in the title to be ineffective for my reading. As I skim over the entire header area to read the "content" area. What I read on this page after the changes is the status message, the bullet list, and the buttons. I miss the header title entirely.

So I think a solution that would solve the lonely bullet list and my title problem would be keeping the title "Confirm uninstall" and moving the question sentence into the body area.

@klonos
Copy link
Member Author

klonos commented Jul 31, 2018

I have updated the PR. Better now?:

screen shot 2018-07-31 at 2 34 18 pm

@opi
Copy link

opi commented Jul 31, 2018

Yay! Marking as RTBC.
@quicksketch please squash the 3 commits before merge to keep a clean git history.

@klonos
Copy link
Member Author

klonos commented Jul 31, 2018

@quicksketch always squashes 😄

image

image

@quicksketch
Copy link
Member

Thanks folks! Merged backdrop/backdrop#2220 into 1.x. Though this could be included in 1.10, as we're so close to 1.11 it's better to avoid unnecessarily changing strings used in translations.

@quicksketch
Copy link
Member

Also, per @herbdool's PR comment, I reverted the title change. It had been changed to "Confirm uninstallation", I reverted it back to "Confirm uninstall". The body and message area is the big win here.

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

4 participants