-
Notifications
You must be signed in to change notification settings - Fork 34
fix: Resolve WAVE accessibility violations for WCAG 2.1 AA compliance #407
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
Open
Sagar-6203620715
wants to merge
17
commits into
cloudnative-pg:main
Choose a base branch
from
Sagar-6203620715:test-all-ossf-fixes
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
fix: Resolve WAVE accessibility violations for WCAG 2.1 AA compliance #407
Sagar-6203620715
wants to merge
17
commits into
cloudnative-pg:main
from
Sagar-6203620715:test-all-ossf-fixes
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…yles for WCAG AA fixes and improved keyboard navigation across header, footer, hero cards Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…els, and remove duplicate responsive headings across homepage sections Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…ve redundant logo link to reduce WAVE alerts and improve screen reader experience Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…and fix meta description handling for improved screen reader navigation and document structure Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…ve keyboard accessibility and semantic structure Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…to meet WCAG AA contrast requirements Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…essibility and semantic structure Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…d enable .sr-only focus visibility to complete accessibility corrections Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…, and update nav link contrast for improved accessibility Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…onvert mobile menu to button, and update nav link contrast for complete accessibility compliance Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
…er link accessibility Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
a11y): Fix contrast errors and possible heading alert - Changed '100%' from <p> to <h2> to fix 'Possible heading' alert and improve contrast (white on blue instead of red on blue) - Changed 'Apache 2 license' and 'GitHub' links from text-rose-600 to text-white underline to meet WCAG AA contrast requirements on blue background - Fixes WAVE errors: 3 contrast errors and 1 possible heading alert Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
- Changed --color-accent-red from #DC2626 to #B91C1C to ensure 5.74:1 contrast ratio with white text - Fixes WAVE contrast errors in feature cards section (white text on bg-red-1) - Also improves hero CTA button contrast - Fixes WAVE errors: 15 contrast errors Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
- Updated bg-red-1 fallback color to match CSS variable (#B91C1C) - Added font-semibold to card body text to qualify as large text (3:1 requirement instead of 4.5:1) - Fixes WAVE contrast errors in feature cards section (Autopilot, Data persistence, Designed for Kubernetes) - Fixes WAVE errors: 15 contrast errors Signed-off-by: Sagar-6203620715 <sagar6203620715@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR addresses all WAVE accessibility violations identified in issue #389, bringing the site to WCAG 2.1 AA compliance. Key improvements include: fixing 15+ contrast errors by updating color tokens and footer/feature card text colors; implementing skip link and accessible focus indicators; converting mobile menu to keyboard-accessible button with ARIA attributes; normalizing heading hierarchy (single h1, proper h2/h3 levels); adding semantic landmarks and sectioning elements; fixing decorative image alt text; and improving meta descriptions. All changes maintain visual design while ensuring accessibility for screen readers and keyboard users.
Fixes #389
score increased