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 197 forks source link

508-defect-2: Military service history needs design review (and possible redesign) #48261

Closed joshkimux closed 3 months ago

joshkimux commented 1 year ago

What happened?

The military service history needs a design review as:

For more on a11y problems with the list loop pattern, visit this ticket from coe.

https://user-images.githubusercontent.com/14154792/195929366-db11f5e8-4152-468f-a786-50a2e711109a.mov

Screen Shot 2022-10-14 at 3 48 15 PM

Steps to Reproduce

Desired behavior

Acceptance Criteria

How to configure this issue

sonntag-adam commented 1 year ago

@ccruzgra @barron-charles

ccruzgra commented 1 year ago

@joshkimux In the desired Behavior where you state secondary research needs to be conducted to figure out what needs to be happening, is that Platform folks that need to do that or does that fall on my team?

joshkimux commented 1 year ago

@ccruzgra this would fall within your team 🙇

MichelleDieudonne commented 1 year ago

Jira work item link: https://vajira.max.gov/browse/MBMS-37682

ccruzgra commented 11 months ago

@coforma-terry Could you please clarify the issue above for it's missing unique labels for list loop buttons. Is that referring to the aria labels missing from the card's buttons?

Also, bullet 2 regarding EVSS does not apply to us as our Tech Lead confirmed we don't use EVSS.

coforma-terry commented 11 months ago

Hi @ccruzgra - let me check with one of the a11y specialists, this one's out of my wheelhouse :)

eli-oat commented 11 months ago

@ccruzgra thanks for this question!

this ticket provides more details about the list loop buttons. I am new to this specific ticket, so am not certain of the markup you are working with (is there a link to this in staging I could review?), but the tl;dr is that when building a list of distinct but similar data you need someway to distinguish between the duplicate date other than visually. Typically this is resolved with aria, but it could be done with heading grouping, I believe (I'd want to confirm that with DST before suggesting that as a diffinitive answer).

ccruzgra commented 11 months ago

Hi @eli-oat. This form is found at https://staging.va.gov/burials-and-memorials/pre-need/form-10007-apply-for-eligibility/introduction.

barron-charles commented 11 months ago

@eli-oat did you have a chance to speak with DST regarding this issue?

eli-oat commented 11 months ago

I haven't yet spoken with them, @barron-charles

Sorry, I was speaking in the hypothetical and wasn't sure how you all wanted to proceed. I will reach out to them now and follow up.

barron-charles commented 11 months ago

Thanks you! It's much appreciated.

eli-oat commented 11 months ago

This has been a difficult one to determine definitively since there are a couple of list loop patterns at exist, including:

That said, the issue initially flagged is still present, wherein you cannot clearly navigate items in any way other than visually.

The heading map shows this, too:

Apply Online For Pre-Need Determination Of Eligibility In A VA National Cemetery | Veterans Affairs
1 Apply for pre-need eligibility determination
    2 Step 2 of 6: Military historyWe’ll save your application on every change.
        3 New service period
    2 Need help?
    2 Veteran programs and services
    2 More VA resources
    2 Get VA updates
    2 In crisis? Talk to someone now
    2 Get answers
    2 Call us
    2 Visit a medical center or regional office
    2 Language assistance
Screenshot 2023-09-19 at 10 44 03 AM

When the heading map is compared to that screenshot of the form, you'll notice that the previous military history item isn't present -- this is similar to the experience in voice over, where it is difficult to navigate to the existing military history.