Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Main nav updates #581

Closed
7 of 16 tasks
r-bartlett-gsa opened this issue Dec 16, 2022 · 12 comments · Fixed by #661
Closed
7 of 16 tasks

Main nav updates #581

r-bartlett-gsa opened this issue Dec 16, 2022 · 12 comments · Fixed by #661
Assignees

Comments

@r-bartlett-gsa
Copy link
Contributor

r-bartlett-gsa commented Dec 16, 2022

As a benefits tool user, in order to navigate between USAGov and benefits tool subsite without confusion, I would like the benefit tool subsite to resemble USAGov look and feel as close as possible.

Acceptance criteria:

  • Update main nav to match USAGov's beta site (no mega menu)

Resources:

Definition of Done:

  • Code complete
  • Tests coverage is greater than team benchmark (90% goal)
  • Security scans passed
  • Acceptance Criteria is met and it works as expected
  • Accessibility tested
  • Cross browser tested
  • Build process and deployment is automated and repeatable
  • Load testing/performance testing
  • Self Documentation whenever possible
  • Feature toggles if appropriate
  • Deployed to staging
  • Usability testing
  • PR approved / Peer reviewed
  • A11y tested by Meghan
  • PO approved (UX, usability, edge cases)

More details about each item can be find in BEARS P4 DoD google doc

QA checklist

@fpigeonjr
Copy link
Contributor

fpigeonjr commented Feb 6, 2023

This is part of the work on PR #616.

This was referenced Feb 7, 2023
@danny-englander
Copy link
Contributor

This is dependent on #580 and #582.

@fpigeonjr
Copy link
Contributor

I have completed my involvement with this GitHub issue and am stepping away from it.
Thank you for your understanding.

@danny-englander
Copy link
Contributor

danny-englander commented Mar 10, 2023

@nehemiah-abuga Ready for testing.

  • Pull multi-life-event-2023 to your local
  • run npm i
  • Run npm run dev
  • Go to http://localhost:3000
  • Inspect the main navigation and ensure that the link text and links match what is on https://beta.usa.gov/
  • Switch language to Spanish
  • Inspect the main navigation and ensure that the link text and links match what is on https://beta.usa.gov/es

@nehemiah-abuga
Copy link
Contributor

@danny-englander - Beta.usa.gov has Travel while local:3000 has immigration and U.S. citizenship. Please let me know if this is the expected functionality

@danny-englander
Copy link
Contributor

@nehemiah-abuga, I have adjusted the navigation, both languages, to match beta.usa.gov (again). Noting that they keep changing the navigations on a regular way so without an API, unfortunately we will not always be up to date with beta.usa.gov.

@danny-englander danny-englander removed their assignment Mar 20, 2023
@nehemiah-abuga
Copy link
Contributor

I have tested this and verified the changes

@danny-englander
Copy link
Contributor

@amycole501
Copy link

  1. Keyboard testing appeared to be fine.
  2. When zooming in 200% on Chrome, the menu button appears after shifting to a mobile layout. The button in Chrome almost appears to be cut off on the right. I'm not sure it's an error per se but it looks almost like it's being covered up by the right scroll bar on my screen. let me know if you need a screen shot.
  3. When tabbing into the nav from the top of the page and when listening in JAWS I noticed the order of the elements is off. I am taken from the USA.Gov logo straight into the nav. Ideally the person would go to the right into the Phone number and search area since those appear above the nav.
  4. Make sure to note that it's the primary navigation. I didn't hear it announced in JAWS. The USWDS has their site coded with the correct markup to hear "Primary navigation" in screen readers. https://designsystem.digital.gov/components/overview/

@danny-englander danny-englander self-assigned this Mar 27, 2023
danny-englander added a commit that referenced this issue Mar 30, 2023
danny-englander added a commit that referenced this issue Mar 31, 2023
danny-englander added a commit that referenced this issue Mar 31, 2023
danny-englander added a commit that referenced this issue Apr 3, 2023
danny-englander added a commit that referenced this issue Apr 4, 2023
@danny-englander
Copy link
Contributor

danny-englander commented Apr 4, 2023

@amycole501 I have addressed all the issues now except item # 3. Ernie said to pass on that as we are going to put all our efforts into V2 for the most part.

https://federalist-edd11e6f-8be2-4dc2-a85e-1782e0bcb08e.sites.pages.cloud.gov/preview/gsa/usagov-benefits-eligibility/multi-life-event-2023/death-of-a-loved-one/

@danny-englander danny-englander removed their assignment Apr 4, 2023
@amycole501
Copy link

In JAWS the language button says it's both a banner region and a navigation region. I'm not sure which it should be but I think it's banner. May warrant a minor fix.

The header does now read correctly as "header primary navigation region".

@danny-englander
Copy link
Contributor

@amycole501 Can you open a new ticket for a followup on the comment above so we can track it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants