-
Notifications
You must be signed in to change notification settings - Fork 1
Feature audit: Facility Locator #32
Comments
@gnakm @NatalieEMoore I'm working through an audit of all the new features we've released to capture new UI patterns and manage consistency. Would love your feedback on the inconsistencies / for discussion sections above. I haven't looked at any of the existing FL bugs, so some of these may have already been captured in issues. Thanks! |
@melwoodard 1-8 for bugs looks correct, they are bugs. I can add separate issues on the vets.gov/website repo if that makes the most sense @gnakm? for discussion, # 5, I do think the disclaimer should be the same across the site so adding yellow may be a good idea. |
@melwoodard Thank you for another thorough audit. I filed most of the bugs. Please check my thoughts in bold! Inconsistencies / bugs
For discussion
We actually tried it without the filled in color and there wasn't enough contrast, especially with the map on desktop. Not sure if this has to align with secure messaging. I see it more as each tool need slightly different treatment for visual balance on the page.
Another thing we went back and forth on but glad you flagged it now that we have more tools.
I think we should get rid of back link the way we aiming to do for Rx and add title of Facility location in breadcrumb. @melwoodard Let me know what you think and I'll file an issue. Yes it should! Filed issue here
Context: We went back and forth on this because when we had it as the blue information alert box, users missed it. It's actually an on going problem we have with the blue information call out box all across Vets.gov. A handful of us agreed that the white box was a lot easier to see but we actually never tested it. @NatalieEMoore I think we talked about how this is not really an alert but informational. I defer to you. If yellow box makes more sense, I will file an issue!
Agreed! I think Ben was using the same components for results listing for detail page but we should definitely tweak this. Filed issue here |
Thanks Gina!
|
Update: @gnakm ok, I'm gaining some sympathy for the outlined box :) I'm finding a similar scenario in the alert we're adding for signing into Rx/SM about needing an MHV account. A yellow alert feels too strong here, and potentially annoying since a returning user could use this path regularly to get to the Rx / SM apps (rather than sign in through the nav). Thoughts? |
For number 5: Gina wrote, "Context: We went back and forth on this because when we had it as the blue information alert box, users missed it. It's actually an on going problem we have with the blue information call out box all across Vets.gov. A handful of us agreed that the white box was a lot easier to see but we actually never tested it. @NatalieEMoore I think we talked about how this is not really an alert but informational. I defer to you. If yellow box makes more sense, I will file an issue!" Can we do testing? I think it's both an alert and information. How is it being shown across the site currently? @gnakm @melwoodard |
I think testing would be valuable - another thing to think about in the case of FL is the placement of the alert. I can see how it could easily be dismissed as an ad or something irrelevant when it's toward the end of the content. We're using the blue background style across the site, in two flavors - a short and sweet banner with the (i) icon used for brief alerts, and a lengthier callout used to explain eligibility for a particular benefit. Alert: Eligibility: |
Notes from usability testing when we tested with blue box: |
@melwoodard How about using the white box with outline for informational call outs? I think they stand out more and I agree, looks less like an ad. Current: Potential: I think leaving the blue for longer content works because it works better in a long body of text. It's more a matter of making sure that we have consistent information in those boxes (i.e. are you eligible? Who is covered). |
New patterns
1. Custom dropdown with legend icons
2. Disabled text field state
3. Boundary around an application 'chrome'
4. Outline alert message style
5. Contact links with icons
6. Self-contained scrolling list
Inconsistencies / bugs
Page headings are
h3
- should beh1
Dropdowns are missing ^ carat when opened
h4 highlight
classFor discussion
The text was updated successfully, but these errors were encountered: