frontend: Fix register AKS cluster heading#233
Conversation
There was a problem hiding this comment.
Pull request overview
This PR aims to resolve a Lighthouse accessibility finding in the “Register AKS Cluster” modal by correcting the semantic heading hierarchy to better match the visual styling and improve screen reader navigation.
Changes:
- Updated the modal title markup to adjust the rendered heading element for “Register AKS Cluster”.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
vyncent-t
force-pushed
the
fix-aks-register-heading
branch
from
8513bb2 to
2d316bf
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
vyncent-t
force-pushed
the
fix-aks-register-heading
branch
from
2d316bf to
db756ca
Compare
Signed-off-by: Vincent T <vtaylor@microsoft.com>
vyncent-t
force-pushed
the
fix-aks-register-heading
branch
from
db756ca to
8dd2d69
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
This PR fixes an a11y issue identified by Lighthouse where headings were not in sequential order for the "Register AKS cluster" modal.
The title component was visually styled as an h6 header with no larger headers present on the screen. This caused Lighthouse to flag improper heading hierarchy which can negatively impact screen reader navigation.
Type of Change
Related Issues
Closes #193
Related to #219
Changes Made
How to test:
Images