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

Improve discoverability of “Save Changes” button on AMP settings screen #5353

Closed
westonruter opened this issue Sep 8, 2020 · 12 comments · Fixed by #5443
Closed

Improve discoverability of “Save Changes” button on AMP settings screen #5353

westonruter opened this issue Sep 8, 2020 · 12 comments · Fixed by #5443
Labels
UX WS:UX Work stream for UX/Front-end
Projects
Milestone

Comments

@westonruter
Copy link
Member

Feature description

Given a viewport sized as following when looking at the AMP settings screen:

image

Notice there is no Save button visible. This is even worse when a Reader theme is selected.

In the same way that there is a fixed bar at the bottom in the onboarding wizard, there may need to be something similar for the settings screen so that users can discover that they need to click “Save Changes”, without having to scroll all the way down the page.


Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

Implementation brief

QA testing instructions

Demo

Changelog entry

@westonruter westonruter added UX WS:UX Work stream for UX/Front-end labels Sep 8, 2020
@westonruter westonruter added this to Backlog in Ongoing via automation Sep 8, 2020
@jwold
Copy link
Collaborator

jwold commented Sep 8, 2020

So something like this?

Settings

In addition to our own onboarding, I found a few other examples from Yoast and WooCommerce that are similiar.

@westonruter
Copy link
Member Author

Yes, like that.

@jwold jwold removed their assignment Sep 9, 2020
@jwold
Copy link
Collaborator

jwold commented Sep 17, 2020

Right now the gray Save changes button stays at the bottom of the screen. Users may miss it as they open the panels and interact with settings.

Before: https://d.pr/i/70O8mw

After: https://d.pr/i/ncOO9D

Similar to the onboarding wizard, this would make the gray button float along the footer, so that a user could see it at all times.

This follows patterns that other major plugins are using, and could simplify the settings page as users expand the larger panels.

cc @amedina

@westonruter
Copy link
Member Author

Nice thing about this is the user can hit save, then switch to a different tab to see the result. If they don't like it, they can switch back to the first tab and change the selection and re-save, all without changing the vertical scroll position.

@westonruter westonruter moved this from Backlog to To Do in Ongoing Sep 17, 2020
@amedina
Copy link
Member

amedina commented Sep 17, 2020

I love that approach.

When I was creating the video content, that trip to the basement of the page was painful.

@johnwatkins0 johnwatkins0 added this to the v2.1 milestone Sep 18, 2020
@johnwatkins0
Copy link
Contributor

@jwold How about the save success notice? Right now it flies in from the side. Should we instead work that information (and the rarer save failure notice) into this fixed bottom bar somehow?

@westonruter
Copy link
Member Author

Yes +1

@jwold
Copy link
Collaborator

jwold commented Sep 22, 2020

Ok, here's an idea I've been playing with.

Screen Shot 2020-09-22 at 1 28 53 PM

  1. The initial state is blue and just has the word Save
  2. During the save process (similar to Gutenberg) it will switch to say Saving
  3. When save is complete it will say Save, until changes are made on the page
  4. If an error occurs it will say Save Error, and display text beside that describes the error

Figma: https://www.figma.com/file/SfMlDvHc5KHxJmAZN12PIM/AMP-Design?node-id=1089%3A13117

@westonruter
Copy link
Member Author

@jwold I don't think the button should become the notice, as otherwise what will the user click to retry? Clicking “Save Error” seems to indicate that the error would be saved. I think the success and error notices should be floated to the right.

@jwold
Copy link
Collaborator

jwold commented Sep 24, 2020

You're right! That makes more sense, and it follows with the pattern above with notices above the template modes.

Screen Shot 2020-09-24 at 9 07 59 AM

@johnwatkins0
Copy link
Contributor

👍 This will work. I can use the "Saved" state, which will revert back to "Save changes" once the user makes additional changes.

@johnwatkins0 johnwatkins0 moved this from To Do to In Progress in Ongoing Sep 24, 2020
@johnwatkins0 johnwatkins0 assigned johnwatkins0 and unassigned jwold Sep 28, 2020
@johnwatkins0 johnwatkins0 moved this from In Progress to Ready for Review in Ongoing Sep 29, 2020
@johnwatkins0 johnwatkins0 moved this from Ready for Review to Ready for QA in Ongoing Oct 21, 2020
@jwold
Copy link
Collaborator

jwold commented Apr 23, 2021

Confirmed in QA! Tested online and offline states.

@jwold jwold moved this from Ready for QA to QA Passed in Ongoing Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UX WS:UX Work stream for UX/Front-end
Projects
Ongoing
  
QA Passed
Development

Successfully merging a pull request may close this issue.

4 participants