department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
281 stars 201 forks source link

Design Intent - Accessibility Feedback - AE Profile - Enhanced Military Info #73138

Closed briandeconinck closed 8 months ago

briandeconinck commented 8 months ago

VFS actions

Thoughts/questions

-

Feedback

Practice areas will document their feedback on the VFS-provided artifacts following the Must, Should, and Consider Framework. Platform Governance reviewers may also provide additional notes that don’t comment on the artifacts themselves but are important for implementation (eg. engineering/coding notes).

andaleliz commented 8 months ago

@briandeconinck thank you so much for this feedback. I have a question about the must item. If we switch to the secondary button, we'll need to use a less specific button label. Something like "Show 5 more military occupational specialties" would be so long!

IMO it's a bit problematic if the label says "Show more (5)", especially when we add another section for a period of service, such as countries. This is a quick-n-dirty mock-up, so please ignore the data, but should give you an idea of what I'm talking about.

Do you agree, that it's problematic?

Regarding the other feedback:

briandeconinck commented 8 months ago

Ugh yeah that's not ideal. I'd like to think there might be a content solution, like "Show 5 more specialities" and "Show 5 more locations," finding the right set of words to make it work and still be concise. But eventually you might run into one that can't be done cleanly like that. I don't think there's anything in the design system that precludes button text from wrapping to the next line, but I agree that "Show 5 more military occupational specialties" is going to be visually rough.

A good-enough-but-not-great solution is to use the button text in your mock-up and add an aria-label to each button to make them distinct for screen reader users. A make-Matt-upset solution would be to explore this as a place for a tertiary button style.

andaleliz commented 8 months ago

Thanks @briandeconinck, glad to know I'm not crazy thinking that is a pretty un-ideal label situation.

I'm hopeful there is a content solution that we can come up with post-research, once we hear how veterans talk about these specialties in their own words. There are so many unknowns! For now, I will use the button label we have in our mock-ups for research, and plan on a better content option emerging from that. I'll keep the aria-label solution in my pocket has an emergency plan, but I don't think it'll be needed.

Btw, I desperately wish I was the kind of designer that could come up with an accessible tertiary button option! This gap in the DS and my skillset has been haunting me since I joined the profile team 3 years ago 😭 😆 I did try using the tertiary button that the online scheduling team has proposed in this mock-up, but I know it's not really ready for teams to use it yet. I'm hoping all-caps won't be required once they do release it into the wild :)