fix: a11y changes for pathfinding control contrast - #3224
Conversation
current light-theme contrast does not meet WCAG AA 4.5:1 for the "X", "Add Destination", and drag and drop icons introduced by multi-destination pathfinding. This addresses that by swapping text-neutral to text-light / text-main.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe pathfinding search controls now use semantic ChangesPathfinding search controls
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This small UI change improves light-theme contrast for pathfinding controls without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Current light-theme contrast does not meet WCAG AA 4.5:1 for the "X", "Add Destination", and drag and drop icons introduced by multi-destination pathfinding. This addresses that by swapping text-neutral to text-light / text-main.
Description
Small change from text-neutral to text-light / text-main on the previously mentioned elements to enhance contrast and meet WCAG AA requirements.
Motivation and Context
Resolves BED-9546
Why is this change required? What problem does it solve?
How Has This Been Tested?
Tested locally in CE
Screenshots (optional):
Types of changes
Checklist:
Summary by CodeRabbit