🎨 Palette: Accessible package selection checkboxes#18
Conversation
Replaced non-semantic div-based selection indicators with hidden but functional native checkboxes in the MyPackages view. This ensures screen reader compatibility and enables keyboard navigation (Tab focus and Space/Enter to toggle) while maintaining the existing custom visual theme. Added unit tests to verify selection logic and accessibility. Co-authored-by: realgarit <183757313+realgarit@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
- Replaced non-semantic selection indicators with accessible checkboxes in MyPackages. - Implemented native indeterminate state for header checkbox. - Restored parent-hover feedback for selection indicators. - Fixed CI failure by skipping deploy step when RENDER_API_KEY is missing. - Added unit tests for package selection. Co-authored-by: realgarit <183757313+realgarit@users.noreply.github.com>
…760761998179 and resolve conflict by keeping .github/workflows deleted
realgarit
temporarily deployed
to
palette/accessible-package-selection-2679144760761998179 - intuneforge PR #18
Render
Destroyed
realgarit
deleted the
palette/accessible-package-selection-2679144760761998179
branch
1 participant
This PR enhances the accessibility of the package selection interface in the "My Packages" view.
💡 What:
divelements used for row/card selection with semantic<input type="checkbox">elements.aria-labelto each checkbox for screen reader clarity.tests/packages.test.tsxto ensure functionality and accessibility.🎯 Why:
The previous implementation was not accessible to keyboard-only users or screen readers, as it relied on
onClickhandlers on non-interactivedivelements.♿ Accessibility:
aria-labelattributes provide context for screen reader users.PR created automatically by Jules for task 2679144760761998179 started by @realgarit