Altinity / Altinity/altinity-sql-browser
Accessibility pass: ARIA on menus/autocomplete/tabs, aria-live on errors, visible focus rings
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 8
- Forks
- 2
- Merge medio
- 1 h 34 min
- PR fusionados (30 d)
- 6
Descripción
Part of #68 (Roadmap to 1.0.0) — should-have. The substantial one.
Problem
The app has almost no accessibility affordances — ~11 a11y attributes across all of src/ (mostly tabindex, one aria-hidden). For a complex SQL IDE this is a significant screen-reader and keyboard-navigation gap.
Scope
- ARIA semantics on the custom widgets so assistive tech announces them correctly:
- autocomplete dropdown →
listbox/option+aria-activedescendant(src/ui/editor-complete.js) - File menu and the user/save popovers →
menu/menuitem(src/ui/file-menu.js,app.jsanchoredPopover) - query tab strip →
tablist/tab(src/ui/tabs.js)
- autocomplete dropdown →
aria-liveon the toast / error region so query errors and "session expired" are announced, not just shown (src/ui/toast.js).- Visible keyboard-focus rings everywhere
outline:noneis set without a replacement (src/styles.css:158,165,320,408,454,528,535,…). Today, tabbing through some controls shows no focus indicator.
Acceptance
- Screen reader announces the autocomplete, menus, and tabs with correct roles/state.
- Errors/toasts are announced via
aria-live. - Every interactive control shows a visible focus ring on keyboard focus.
-
npm testgreen at the per-file coverage gate.
Largest of the 1.0 should-haves; worth its own focused pass + a screen-reader/keyboard test.
Folded in from the roadmap (#68) nice-to-haves:
- Honor
prefers-reduced-motion— a@media (prefers-reduced-motion: reduce)block that disables/limits transitions and animations (chart updates, hovers) for users who've set reduce-motion in their OS.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza leyendo src/ui/editor-complete.js, src/ui/file-menu.js, app.js, src/ui/tabs.js, src/ui/toast.js y las reglas outline:none indicadas en src/styles.css. Ejecuta npm test para entender el umbral de cobertura por archivo y, después, verifica los criterios de aceptación indicados para el lector de pantalla, el foco del teclado, aria-live y la reducción del movimiento con la prueba de lector de pantalla/teclado mencionada.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- css, javascript
- Área
- accessibility, frontend, testing-qa
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100