From d0dd9b79eca35312cee6c005e1bc99f7a7a58f66 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Tue, 30 Jun 2026 22:03:14 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20Replace=20symbolic=20?= =?UTF-8?q?navigation=20hints=20with=20text=20for=20screen=20reader=20acce?= =?UTF-8?q?ssibility?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .jules/palette.md | 3 +++ src/cli/ui/components/CommandSuggestionList.tsx | 2 +- tests/integration/prompt_templates.test.ts | 4 +++- 3 files changed, 7 insertions(+), 2 deletions(-) create mode 100644 .jules/palette.md diff --git a/.jules/palette.md b/.jules/palette.md new file mode 100644 index 00000000..84957adc --- /dev/null +++ b/.jules/palette.md @@ -0,0 +1,3 @@ +## 2024-05-24 - Screen Reader Friendly Navigation Hints +**Learning:** For complex command-line selection interfaces or overlays using Ink, screen readers in CLI environments rely on raw text output and fail to announce purely symbolic navigation text (e.g., `↑↓`, `⏎`) properly, causing an accessibility regression. +**Action:** Use visually concise but screen-reader-friendly text (like 'Up/Down', 'Enter') instead of symbols for keyboard navigation hints to ensure accessibility. diff --git a/src/cli/ui/components/CommandSuggestionList.tsx b/src/cli/ui/components/CommandSuggestionList.tsx index 7e66838a..d51fe548 100644 --- a/src/cli/ui/components/CommandSuggestionList.tsx +++ b/src/cli/ui/components/CommandSuggestionList.tsx @@ -116,7 +116,7 @@ export const CommandSuggestionList: React.FC = ({ - ↑↓ nav · ⏎ select · esc close + Up/Down nav · Enter select · Esc close diff --git a/tests/integration/prompt_templates.test.ts b/tests/integration/prompt_templates.test.ts index 21905259..f1360d78 100644 --- a/tests/integration/prompt_templates.test.ts +++ b/tests/integration/prompt_templates.test.ts @@ -36,7 +36,9 @@ describe('Prompt templates', () => { expect(planSystem).toContain('You are SalmonLoop.'); expect(patchSystem).toContain('You are PATCH, a phase-native diff compiler.'); - expect(autopilotSystem).toContain('You are a senior software engineer running in "autopilot" mode.'); + expect(autopilotSystem).toContain( + 'You are a senior software engineer running in "autopilot" mode.', + ); expect(answerSystem).toContain('You are a coding assistant in "answer" mode.'); expect(researchSystem).toContain('You are a research assistant.'); });