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

feat(navigation): enhance hover styles of navigation links #545

Closed
wants to merge 1 commit into from

Conversation

gregtyler
Copy link
Contributor

@gregtyler gregtyler commented Dec 15, 2023

Applies to Primary navigation, Sub navigation and Side navigation.

Use the standard GDS link hover colour to ensure links colour contrast is sufficient for
readability.

Ensure borders follow suit, and ensure they're always on the same side of the element.

Fixes #447

@gregtyler gregtyler requested a review from a team as a code owner December 15, 2023 09:26
@gregtyler gregtyler added the staging:request Add this label to request a staging environment for a pull request label Dec 15, 2023
Copy link

🚀 Deploying to staging environment

Username: staging, Password: moj

@github-actions github-actions bot added staging:active Automatic label added when the PR is on the staging environment and removed staging:request Add this label to request a staging environment for a pull request labels Dec 15, 2023
@gregtyler gregtyler force-pushed the fix-primary-nav-hover branch from 742dacb to cbccda7 Compare December 15, 2023 15:30
@gregtyler gregtyler changed the title fix(primary-navigation): enhance hover styles fo primary navigation links feat(navigation): enhance hover styles of navigation links Dec 15, 2023
@gregtyler gregtyler removed the staging:active Automatic label added when the PR is on the staging environment label Dec 15, 2023
@gregtyler gregtyler force-pushed the fix-primary-nav-hover branch from cbccda7 to 5d6d544 Compare December 15, 2023 15:31
@gregtyler gregtyler added the staging:request Add this label to request a staging environment for a pull request label Dec 15, 2023
Copy link

🚀 Deploying to staging environment

Username: staging, Password: moj

@github-actions github-actions bot added staging:active Automatic label added when the PR is on the staging environment and removed staging:request Add this label to request a staging environment for a pull request labels Dec 15, 2023
@github-actions github-actions bot removed the staging:active Automatic label added when the PR is on the staging environment label Dec 27, 2023
@gregtyler gregtyler mentioned this pull request Jan 25, 2024
Applies to Primary navigation, Sub navigation and Side navigation.

Use the standard GDS link hover colour to ensure links colour contrast is sufficient for
readability.

Ensure borders follow suit, and ensure they're always on the same side of the element.

Fixes #447
@gregtyler gregtyler force-pushed the fix-primary-nav-hover branch from 5d6d544 to 5d28440 Compare January 25, 2024 13:38
@gregtyler
Copy link
Contributor Author

Done in #565

@gregtyler gregtyler closed this Jan 25, 2024
@gregtyler gregtyler deleted the fix-primary-nav-hover branch January 25, 2024 15:43
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.

Colour contrast for Primary navigation hover is not AA compliant
2 participants