Misc Usability Testing Fixes

Automattic / bugomattic

Bugomattic is a tool that guides bug reporters to the right actions within large, complex organizations

GNU General Public License v2.0

6 stars 0 forks source link

Misc Usability Testing Fixes #40

Closed dpasque closed 1 year ago

dpasque commented 1 year ago

What Does This PR Add/Change?

This PR is a grab-bag of the smaller fixes that came out of our usability testing. See pciE2j-1Pp-p2 for context.

Fixes include...

Changing the github task default title to include the repo name.
Adding an example search to the feature search placeholder text.
Adding some error messages to the "Next Steps" section if folks URL smash their way to a bad state, or if there's somehow no tasks configured for a given combination.
Replace the highlighting style in the feature searching with just a bolder font-weight.
Fix the extra scrollbar that appears in the breadcrumb (TIL, overflow-x actually changes the default behavior of overflow-y!)

Testing Instructions

[x] Unit tests pass
[ ] Feel free to yarn start and play around with the changes!

Issues

Related to #
Closes #

john-legg commented 1 year ago

@dpasque this looks good! Lots of great changes 👏

Everything works as expected and I especially like the new highlighting style for feature searching -- it looks way better. I added a few copy-related nitpicks that are super minor, but I'll go ahead and approve it in case you need to move forward with it 🚀

I also had a few ideas I wanted to run by you in a Slack thread, so look out for that soon!

dpasque commented 1 year ago

Alright @john-legg, everything should now be up to date with your design! 👍 (including adding More Info back in). Let me know what other changes and tweaks I should make! 😄

john-legg commented 1 year ago

@dpasque this is awesome! Thanks for making these changes so quickly. I know I threw extra stuff your way today 😬

I finished the accessibility testing and everything works as expected. I especially like how it immediately reads the "missing information" and "no reporting configuration" error messages as soon as it lands on the page 👏

suggestion (non-blocking): Two final tweaks I'd like to suggest are:

Reducing the max-width of .badStateImage just a bit. I put 400px and it looked good.
Adding a margin below the Start Over button on the "missing information" error state. I put 2rem and that seemed about right.

Other than that, I think this PR is ready. Feel free to merge whenever you want 🚀

dpasque commented 1 year ago

Thanks for making these changes so quickly. I know I threw extra stuff your way today 😬

@john-legg thank YOU for all the great feedback as always -- these look sooooooo much nicer now! I really appreciate all your input, never hold back, please! 😁