Vet Center incorrect heading structure in accordions

[laflannery]

Description

At the bottom of all Vet Center pages, there are 2 heading structure issues that should be resolved:

1. The first issue is that all accordion titles should be h3s but the "How we are different" section has accordions that are h2s:
2. Second issue is, within this particular accordion, there is an alert that has an h2 heading. This should actually be an h4:

Question

I would also make the case that this alert isn't necessary, this content can simply be added as a basic list on the page - putting it within the additional info component is adding additional clicks and complexity that isn't needed. Do we have the authority to change this format?

Acceptance Criteria

• [ ] Update the heading of the "Am I eligible for Vet Center services as a Veteran or service member?" accordion to be an H3
• [ ] Consider removing the alert within this accordion and reformatting the content to reduce cognitive load. Be sure that the heading structure is correct.
• [ ] If the alert is not removed, update the alert title to be an H4
• [ ] Requires accessibility review

[Agile6MSkinner]

I'm moving this back to the backlog because it appears that there is design work that still needs to be done before we can point it.

[laflannery]

@mmiddaugh and possibly @aklausmeier In the ticket description I have a question that I think you might be able to provide insight on, I have provided the info/question here as well:

On each Vet center page, in the "How are we different" section, there is an accordion for "Am I eligible for Vet Center services as a Veteran or service member?". This accordion content has an alert with a header level defect currently. However I am curious if we even need to have this content in an alert. It's a simple bullet list and by adding this content in an additional info component within an alert just adds complexity that I don't think we need.

Do you know if we are able to remove this format and simply add the bullets within the content instead?

[aklausmeier]

I am in favor of pulling content out of hidden additional info components and don't see a need for the alert. Unless @mmiddaugh has historical context/prior decisioning for keeping this.

[jilladams]

The current body of this ticket is isolated to 2 things:

1. Fixing accordion headers to become h3
2. Changing the alert header within the eligibility accordion from h2 to h4

Comments from Laura / Amanda in ticket say:

• Remove the alert within that accordion, and convert that content to become bullets, unless @mmiddaugh objects.

I don't see any design calls that need to be made. @mmiddaugh if you can advise about the Alert, we can update ticket and refine this. Moving to Backlog until we get that clarified.

Also noting: this ticket isn't currently in an epic. I don't think it makes sense to tack it onto Vet Center 2.0, so didn't do that.

[laflannery]

@Agile6MSkinner Tagging you because I would like to talk about this in our meeting tomorrow and this way we can easily find it