Skip to content

Various UX cleanup from user testing sessions.#234

Merged
jgerigmeyer merged 19 commits intomasterfrom
ux-cleanup
Mar 30, 2020
Merged

Various UX cleanup from user testing sessions.#234
jgerigmeyer merged 19 commits intomasterfrom
ux-cleanup

Conversation

@jgerigmeyer
Copy link
Copy Markdown
Member

@jgerigmeyer jgerigmeyer commented Mar 27, 2020

Cute animal pic

Because everyone likes pictures of animals.

wombat

Link to Trello card (if applicable)

If this PR relates to a Trello card, don't forget to reference this PR on the card as well.

https://trello.com/c/yqKk4D4Q

Description

The commit messages say what you did; this should explain why and how.

  • Description is in Trello card

Various pieces of cleanup coming out of user testing sessions.

Impacted areas of website

Provide URLs and/or a description of where to look.

  • Capture changes modal, help text for directories is moved into tooltips:

Screenshot 2020-03-27 13 10 51

  • GitHub users are sorted alphabetically
  • Split "Make changes in Dev Org" and "Retrieve changes from Dev Org" into two steps

Screenshot 2020-03-28 14 59 19

  • Progress bar no longer looks clickable
  • Cannot create Test Org if there are no task commits
  • "Create Org" btns in cards are primary when that step is active

Screenshot 2020-03-28 15 06 04

  • "Review" -> "Test", "Reviewer -> "Tester", "Review Org" -> "Test Org"
  • "Capture Changes" -> "Retrieve Changes"
  • Success toasts disappear when another success toast is added
  • Show "Check for Unretrieved Changes" as large btn when no known changes to retrieve

Screenshot 2020-03-28 15 08 49


Reviewer tasks:

  • Pulled branch and ran code locally (or viewed on preview deployment, if applicable)
  • Reproduced the issue to review in the browser
  • Reviewed code
  • Updated Trello card (if applicable)

jgerigmeyer and others added 6 commits March 27, 2020 13:06
@jgerigmeyer
Copy link
Copy Markdown
Member Author

@jgerigmeyer jgerigmeyer marked this pull request as ready for review March 28, 2020 02:39
@jgerigmeyer jgerigmeyer merged commit 984ce54 into master Mar 30, 2020
@jgerigmeyer jgerigmeyer deleted the ux-cleanup branch March 30, 2020 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants