Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DP-30366 focus indicator style for WCAG 2.2 #1846

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

ygannett
Copy link
Contributor

@ygannett ygannett commented Nov 8, 2023

Any PRs being created needs a changelog.txt file before being merged into dev. See: Change Log Instructions

Description

Mockup focus indicator to meet WCAG 2.2 update.

scss/00-base/mixins/_ma-state-focus.scss covers all styles except for the main navigation (both horizontal and flyout).
The main nav styles are modified in:

  • 02-molecules/_main-nav-hamburger.scss
  • 03-organisms/_header-hamburger.scss

Form elements are modified in:

  • 01-atoms/_input-checkbox.scss
  • 01-atoms/_input-radio.scss
  • 01-atoms/_select-box.scss

Related Issue / Ticket

Steps to Test

  1. Review the mockup and decide if they need any adjustments for more visual impact, toning down, or aesthetics.

Screenshots

Use something like licecap to capture gifs to demonstrate behaviors.

Additional Notes:

Anything else to add?

Impacted Areas in Application

@todo

Today I learned...

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

Successfully merging this pull request may close these issues.

1 participant