π¨ Palette: Add clear disabled states and tooltips when offline#87
π¨ Palette: Add clear disabled states and tooltips when offline#87teerthsharma wants to merge 1 commit into
Conversation
Disabled the chat input and send button when `tunnelStatus` is not 'LOCKED'. Updated the input placeholder, title, and ARIA labels dynamically to explain to the user why they cannot send messages when the kernel is offline. This provides clear contextual feedback and improves accessibility. Co-authored-by: teerthsharma <78080953+teerthsharma@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:
Disabled the chat input and send button when the
tunnelStatusis not 'LOCKED' (offline). Updated the input placeholder, button title, andaria-labels dynamically to explain that the kernel is offline and waiting for a connection.π― Why:
Previously, users might try to send messages while the kernel was offline without receiving immediate feedback that their input wouldn't be processed. By visibly disabling the inputs and providing explicit reasons (via placeholders and tooltips) for why they are disabled, we remove confusion and make the system's state much clearer and more intuitive.
πΈ Before/After:
(Visual change: The input field now reads "Kernel offline. Waiting for connection..." and both the input and button appear grayed-out/disabled when offline instead of looking interactive.)
βΏ Accessibility:
Coupled the visual
disabled:opacity-50 disabled:cursor-not-allowedstyles with dynamicaria-labelandtitleattributes on both the input and submit button. This ensures that screen reader users and mouse users hovering over the elements understand precisely why the interactive elements are currently disabled.PR created automatically by Jules for task 13066326533245901586 started by @teerthsharma