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

Update appointment list headings to align with VAOS #58140

Closed cgednalske closed 1 year ago

cgednalske commented 1 year ago

Following a recent sync with VAOS designers, the headings for the appointment list date need to be updated to align with the current VAOS design.

Tasks

Sketch:

CodePen:

Acceptance Criteria

loripusey commented 1 year ago

@cgednalske @benbrasso-agile6 Questions:

benbrasso-agile6 commented 1 year ago

I don't think the current app has this pattern, because we only ever show appointments for "today," and therefore, the design pattern is slightly different. Christina?

cgednalske commented 1 year ago

Yep that's right! This just applies to the appointment list, so the current app is all set as is 👍

cgednalske commented 1 year ago

Hi @loripusey -- @benbrasso-agile6 and I just touched base on this ticket. I have capacity to pull it into into this sprint so it's done before midpoint review. Any updates I should make to the ticket?

cgednalske commented 1 year ago

@loripusey @benbrasso-agile6 this is ready for validation. The appointment list dates have been updated to h4 for both Sketch and CodePen. The annotations page in Sketch has also been updated to include that new heading and aria label. (This took me about 2.5 hours if we want to retrospectively point it.)

benbrasso-agile6 commented 1 year ago

LGTM