Skip to content

USWDS - Pagination: Added stylings for Windows High Contrast mode#4523

Merged
mejiaj merged 1 commit into
cm-high-contrast-a11yfrom
cm-hc-pagination
Feb 22, 2022
Merged

USWDS - Pagination: Added stylings for Windows High Contrast mode#4523
mejiaj merged 1 commit into
cm-high-contrast-a11yfrom
cm-hc-pagination

Conversation

@mahoneycm

@mahoneycm mahoneycm commented Feb 16, 2022

Copy link
Copy Markdown
Contributor

Preview

Pagination →

Description

Resolves #4517 Pagination issues

Adds forced-color stylings for support in Windows High Contrast mode.

Before, the current page did not have depictive stylings so users could not easily see which page they are on, versus what is next or previous. This is mostly a problem in the unbounded variant where there are 4 page numbers grouped together.

Additional information

Default:

image

Unbounded:

image

Current w/ high contrast:

image
image

Fix w/ high contrast:

image
image

Before you hit Submit, make sure you’ve done whichever of these applies to you:

  • Follow the 18F Front End Coding Style Guide and Accessibility Guide.
  • Run npm test and make sure the tests for the files you have changed have passed.
  • Run your code through HTML_CodeSniffer and make sure it’s error free.
  • Title your pull request using this format: [Website] - [UI component]: Brief statement describing what this pull request solves.

@mahoneycm mahoneycm requested a review from mejiaj February 16, 2022 20:39
@mahoneycm mahoneycm linked an issue Feb 16, 2022 that may be closed by this pull request
8 tasks
@mahoneycm mahoneycm mentioned this pull request Feb 16, 2022
8 tasks

@mejiaj mejiaj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and clean, great writeup too. Thank you!

@mejiaj mejiaj merged commit 4dbfb6a into cm-high-contrast-a11y Feb 22, 2022
@mejiaj mejiaj deleted the cm-hc-pagination branch February 22, 2022 21:47
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.

Phase 1: Windows HC Fixes

2 participants