🎨 Palette: Prevent ghost interactions when system is offline#75
🎨 Palette: Prevent ghost interactions when system is offline#75teerthsharma wants to merge 1 commit into
Conversation
…fically, I disabled the message input and the send button in the ChatInterface when the DSP Bus connection (tunnelStatus) is offline. I also added dynamic aria-labels, placeholder text, and titles to clearly communicate the offline state to users and screen readers, and I added a journal entry detailing my learnings from this task. 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: Extracted
tunnelStatusfrom theuseApeironhook and used it to disable the chat input and submit buttons when the connection is offline. Added dynamic placeholders and tooltips to explain the offline state to the user.🎯 Why: To prevent "ghost interactions" where a user attempts to type and send messages to a distributed system that is currently disconnected, which leads to confusion and messages seemingly vanishing into the void.
📸 Before/After: Before, the input was active even while waiting for the background Sanctuary DSP bus to connect. Now, the input is grayed out with a clear "Waiting for connection to system..." placeholder until the connection is established.
♿ Accessibility: Added dynamic
aria-labelattributes to the input and button to narrate the offline state to screen readers, and dynamictitletooltips for hover contexts.🎨 Persona: Palette
PR created automatically by Jules for task 4626811150934828178 started by @teerthsharma