Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
4 changes: 4 additions & 0 deletions .Jules/palette.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
## 2024-05-15 - [Filter Pills Accessibility]

**Learning:** Custom interactive elements like filter pills must have `aria-pressed` for screen readers to recognize them as toggle buttons. Dark theme elements often need explicit `focus-visible` styles (`focus-visible:ring-2 focus-visible:ring-white/50 ...`) because default browser focus rings can be invisible against dark backgrounds.
**Action:** Always add `aria-pressed={isActive}` to toggle elements and explicit dark-theme focus rings to custom buttons.
2 changes: 2 additions & 0 deletions .github/workflows/model-quality.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ jobs:
REAL_DATA_MIN_DAYS: 7
LEARNED_KFOLDS: 5
LEARNED_MAX_VAL_LOSS: 0.2
FORECAST_BACKTEST_STALE_DAYS: 7
FORECAST_BACKTEST_SOURCE: mock
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down
10 changes: 8 additions & 2 deletions src/components/dashboard/filter-pills.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,10 @@ export function FilterPills({ options, selected, onToggle, onClear, className }:
key={option.id}
onClick={() => onToggle(option.id)}
transition={{ duration: 0.15 }}
aria-pressed={isSelected}
className={cn(
'inline-flex items-center gap-2 rounded-full px-3 py-1.5 text-xs font-medium transition-all',
'focus:outline-none focus-visible:ring-2 focus-visible:ring-white/50 focus-visible:ring-offset-2 focus-visible:ring-offset-neutral-900',
isSelected
? 'bg-[#4C8BF5] text-white shadow-sm'
: 'bg-neutral-800 text-neutral-300 hover:bg-neutral-700'
Expand All @@ -60,9 +62,13 @@ export function FilterPills({ options, selected, onToggle, onClear, className }:
{hasSelection && onClear && (
<button
onClick={onClear}
className="inline-flex items-center gap-1 rounded-full px-3 py-1.5 text-xs font-medium text-neutral-400 hover:text-neutral-300 transition-colors"
aria-label="Clear selected filters"
className={cn(
'inline-flex items-center gap-1 rounded-full px-3 py-1.5 text-xs font-medium text-neutral-400 hover:text-neutral-300 transition-colors',
'focus:outline-none focus-visible:ring-2 focus-visible:ring-white/50 focus-visible:ring-offset-2 focus-visible:ring-offset-neutral-900'
)}
>
<X className="size-3" />
<X className="size-3" aria-hidden="true" />
Clear
</button>
)}
Expand Down
Loading