🎨 Palette: [UX improvement] Add Accessibility Info to VS Code StatusBarItem#71
🎨 Palette: [UX improvement] Add Accessibility Info to VS Code StatusBarItem#71
Conversation
This commit improves the accessibility of the SpecGuard VS Code extension by dynamically setting the `accessibilityInformation` property on the CDD Score `StatusBarItem`. Screen readers will now announce human-readable summaries (e.g., "CDD Score: 85 out of 100, Good") rather than attempting to interpret raw strings with codicons like `$(shield) CDD: 85/100 (B)`. The `activate` function was also updated to be explicitly `async` to correctly support `await refreshScore()`. Co-authored-by: raccioly <63126795+raccioly@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. |
💡 What
Added
accessibilityInformationto the VS CodeStatusBarItemto provide meaningful voice feedback for screen readers during initialization and subsequent data refreshes. Theactivatefunction was safely updated toasync.🎯 Why
VS Code
StatusBarItemobjects without explicitaccessibilityInformationread out poorly on screen readers, especially when they utilize visual icons (like$(shield)) or shorthand textual representations (CDD: ?). This ensures developers relying on screen readers receive clear context regarding their Canonical-Driven Development score without visual dependency.📸 Before/After
Before: Screen reader would read the raw text:
"$(shield) CDD: ?"or"$(pass) CDD: 85/100 (B)".After: Screen reader will read:
"CDD Score: Unknown"or"CDD Score: 85 out of 100, Good".♿ Accessibility
role: 'button'to clearly define the UI element interactability.tooltipStatus) and scores to a human-readable spokenlabel.PR created automatically by Jules for task 10616416875242864639 started by @raccioly