A11y Bug 8780801: Fix keyboard focus visibility in high contrast desert theme#609
Conversation
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
|
@copilot Please avoid the use of "!important" |
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
|
@copilot this solution does not work |
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
You're right - the solution without |
|
@copilot there are more issues for desert theme like the icons used are not visible like the smiley for feedback....the cross button for close and the icons used in options panel...please help fix that as well |
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
Fixed the icon visibility issues in commits f0c2fd3 and b2cfc76. Applied drop-shadow filters to all icons using the
The solution uses stronger shadows (2px) for critical UI icons and standard shadows (1px) for other images, creating a glow effect that makes icons visible against any high contrast background. |
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
|
@copilot please remove shadow |
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
This reverts commit fa3c78e.
|
@copilot remove the changes done in package-lock.json |
Co-authored-by: aanchalbhansali <185747873+aanchalbhansali@users.noreply.github.com>
Fix keyboard focus visibility in high contrast desert theme
Summary:
Successfully fixed Bug 8780801 by adding targeted focus indicator styles for high contrast desert theme.
Changes Made:
Focus Visibility Fix:
Added targeted focus indicator styles within the
@media (forced-colors: active)query inclipper.less:a:focus,button:focus,[role="button"]:focus, and.wideActionButton:focusHighlightsystem color with 2px solid outline!importantto override existing focus styles that also use!importantTechnical Details:
Highlightsystem color for focus outlines adapts to active high contrast themeforced-colors: activemedia query is the modern standard for detecting high contrast modeWhy !important is Required:
Existing
.wideActionButton:focusstyles use!important, requiring matching specificity to override.Root Cause:
Focus outline used fixed light color (#f8f8f8) lacking contrast in desert theme.
Testing Results:
Accessibility Compliance:
This fix ensures compliance with MAS 4.3.1 – No Disruption of Accessibility Features by making keyboard focus visible for high contrast users in all themes.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.