From e777c5fad5daed2b40a74d05f7a082572d8c97b9 Mon Sep 17 00:00:00 2001 From: parvezk <3619527+parvezk@users.noreply.github.com> Date: Wed, 22 Jul 2026 07:57:55 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20[UX=20improvement]=20?= =?UTF-8?q?Add=20aria-label=20to=20follow-up=20textarea?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add aria-label to the follow-up message textarea in SessionDrawer for better screen reader accessibility. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> --- .jules/palette.md | 3 +++ web/components/session-drawer.tsx | 1 + 2 files changed, 4 insertions(+) create mode 100644 .jules/palette.md diff --git a/.jules/palette.md b/.jules/palette.md new file mode 100644 index 0000000..45a7dad --- /dev/null +++ b/.jules/palette.md @@ -0,0 +1,3 @@ +## 2024-05-18 - Fix Session Drawer Linting & Accessibility +**Learning:** When migrating a setState from `useEffect` to the render phase for optimistic updates, it's easy to forget to add the `aria-label` to form fields if they just have placeholders. Placeholders are not a substitute for `aria-label` or `id`/`htmlFor` for screen readers. +**Action:** Always check form elements for `aria-label` or `id`/`htmlFor` when reviewing Next.js/React code. diff --git a/web/components/session-drawer.tsx b/web/components/session-drawer.tsx index c0f5a25..db61564 100644 --- a/web/components/session-drawer.tsx +++ b/web/components/session-drawer.tsx @@ -109,6 +109,7 @@ export function SessionDrawer({ value={message} onChange={(e) => setMessage(e.target.value)} placeholder="Send a follow-up to steer the agent…" + aria-label="Follow-up message" className="min-h-20 border-zinc-800 bg-zinc-900" />