About Security Content - Githubissues

GSA / notifications-admin

The UI of Notify.gov

https://notify.gov

Other

11 stars 2 forks source link

About Security Content #2127

Open heyitsmebev opened 3 days ago

heyitsmebev commented 3 days ago

As part of the new About Notify content section, this PR is the implementation for "Security"

Definition of Done

Should include a side navigation
Should be able to easily navigate to About Notify content
Should meet a11y standards

Figma: https://www.figma.com/proto/aAWhlPzOP6kfVFnCCAfSZZ/notify.gov?page-id=2339%3A3817&node-id=5392-875&node-type=frame&viewport=2870%2C-701%2C0.48&t=nfkxOXnOCIFQB1yK-1&scaling=min-zoom&content-scaling=fixed&starting-point-node-id=5392%3A875

Content: https://docs.google.com/document/d/1lP8I0Jxu0B26cmTGBQcoHCbfSvKKHQBE6gzTL9HUheE/edit?tab=t.0#heading=h.icyigvoln8tg

jonathanbobel commented 1 day ago

@heyitsmebev all links that leave the Notify.gov site need an external link icon added https://designsystem.digital.gov/components/link/

Example:

 <a
    class="usa-link usa-link--external"
    rel="noreferrer"
    target="_blank"
    href="https://i.giphy.com/media/WPzQF6ruiIIVzHNlwX/source.gif"
    >external website</a
  >.

dmvancura commented 14 hours ago

Security is a subpage to the About Notify section. It is navigated to after a user selects to view the About Notify section. @CathyBeil and @jonathanbobel, correct, yes? Note: It is in front of the login, but i understand that's a different PR.

heyitsmebev commented 13 hours ago

@dmvancura i revised the menu hierarchy in this PR. Security is indeed a subnav