Skip to content

🎨 Palette: [UX improvement] Add aria-labels to search header buttons #282

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Himan-D wants to merge 1 commit into
base: master
Choose a base branch
from
Open

🎨 Palette: [UX improvement] Add aria-labels to search header buttons #282

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .jules/palette.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,6 @@
## 2026-06-07 - Added aria-labels to icon-only buttons
**Learning:** The project relies on shadcn/ui but has missed several standard accessibility practices like adding 'aria-label' or screen reader only text to icon-only buttons. The Header and Memory Table components, as well as the Sidebar component, have icon-only buttons without proper accessible names.
**Action:** Always verify that 'Button' components with 'size="icon"' and regular '<button>' elements without visible text have 'aria-label' attributes or use '<span className="sr-only">' for accessibility.
## 2026-07-14 - Added aria-labels to ambiguous toggle buttons
**Learning:** Ambiguous controls like toggles with abbreviated text (e.g., 'AI' vs 'Vec' for screen size constraints) require descriptive aria-labels that explain both the current state and available actions.
**Action:** Always add descriptive aria-labels to toggle buttons with abbreviated text to ensure screen reader users understand the component's state and purpose.
2 changes: 2 additions & 0 deletions dashboard/src/components/dashboard/header.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,7 @@ export function Header() {
"h-7 px-2 text-xs gap-1",
searchMode === "spreading" && "bg-primary/90 hover:bg-primary"
)}
aria-label={`Search mode: ${searchMode === "spreading" ? "Spreading Activation (AI)" : "Vector Search"}. Click to change.`}
>
<Zap className="h-3 w-3" />
{searchMode === "spreading" ? "AI" : "Vec"}
Expand Down Expand Up @@ -153,6 +154,7 @@ export function Header() {
setSearchQuery("");
setShowResults(false);
}}
aria-label="Clear search results"
>
<X className="h-3 w-3" />
</Button>
Expand Down
Loading