🎨 Palette: [UX improvement] Context-aware status bar tooltips and state cleanup#93
🎨 Palette: [UX improvement] Context-aware status bar tooltips and state cleanup#93
Conversation
Adds explicit context-aware tooltips to the VS Code extension status bar when the CDD score parsing fails or encounters an error, explaining why the "CDD: ?" state is shown. Also explicitly clears any lingering background colors so that previous failing (red) or passing (green) states don't incorrectly persist during error states. Fixes syntax error where `await` was incorrectly used in the synchronous `activate` function. 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
backgroundColoron the status bar item (statusBarItem.backgroundColor = undefined) during these error states.function activate(context)toasync function activate(context)to resolve aSyntaxErrorcaused by usingawaitinside a synchronous function.🎯 Why
When the extension encounters an error reading the CDD score, it gracefully falls back to displaying
$(shield) CDD: ?. However, previously:?but remain red, sending confusing, mixed signals.📸 Before/After
Before: Status bar shows
CDD: ?but might be styled red/yellow from previous runs, and hovering gives generic static text.After: Status bar shows
CDD: ?with the default background color, and hovering explains why the state could not be refreshed.♿ Accessibility
Improves cognitive accessibility by ensuring developers aren't relying on stale visual cues (lingering red/yellow colors) when the actual state is "unknown." Descriptive tooltips provide clear, readable context for the error rather than requiring users to manually check output logs.
PR created automatically by Jules for task 3474951075863041236 started by @raccioly