🎨 Palette: Add ARIA labels to canvas context menu layout buttons - #244
🎨 Palette: Add ARIA labels to canvas context menu layout buttons#244endlessblink wants to merge 4 commits into
Conversation
Added missing `aria-label` attributes to the layout-related icon-only buttons (Align Left, Arrange in Row, etc.) in `CanvasContextMenu.vue`. The attributes are matched to their existing `title` values to provide a descriptive fallback for screen readers and better general accessibility. Co-authored-by: endlessblink <37155282+endlessblink@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. |
…ntextMenu layout buttons Fix lint and TypeScript errors that caused CI checks to fail: - Fixed `App.vue` `any` warnings by typing `e` as `unknown` and applying the correct narrow typings. - Corrected custom event casing in `QuickSortCard.vue` (`swipe-right` -> `swipeRight`, etc.). - Removed unused `weeklyPlanPrimaryTaskIds` and `weeklyPlanRelatedChipIds` variables from `ChatMessage.vue`. - Added missing `aria-label` attributes to the layout-related icon-only buttons in `CanvasContextMenu.vue`. Co-authored-by: endlessblink <37155282+endlessblink@users.noreply.github.com>
Fixed several CI build failures: 1. `App.vue`: narrowed `any` types to `unknown` and correctly typed the Service Worker `MessageEvent` data. 2. `QuickSortCard.vue`: Fixed vue emit casing from `swipe-right` to `swipeRight` to match standard Vue 3 camelCase guidelines. 3. `ChatMessage.vue`: Removed unused `weeklyPlanPrimaryTaskIds` and `weeklyPlanRelatedChipIds` variables. 4. `CalendarHeader.vue`: Fixed `vue/prefer-true-attribute-shorthand` by using `close-on-click-outside` instead of `:close-on-click-outside="true"`. Co-authored-by: endlessblink <37155282+endlessblink@users.noreply.github.com>
Removed `isScheduledOnCalendar` from `InboxFilters.vue` to fix the ESLint unused variable error breaking CI checks. Co-authored-by: endlessblink <37155282+endlessblink@users.noreply.github.com>
1 participant
🎨 Palette: Added
aria-labelattributes to icon-only buttons in the canvas context menu for improved accessibility.💡 What
Added
aria-labelattributes to the layout-related icon-only buttons inCanvasContextMenu.vue.🎯 Why
These buttons represent significant layout actions (e.g., Align Left, Arrange in Grid) but previously only provided visual
titlehints on hover, which assistive technologies interpret inconsistently. Explicitaria-labelattributes fix this.♿ Accessibility
Improved screen reader announcement for icon-only action buttons.
PR created automatically by Jules for task 1481591983212492739 started by @endlessblink