Altinity / Altinity/altinity-sql-browser

Accessibility pass: ARIA on menus/autocomplete/tabs, aria-live on errors, visible focus rings

Offen
#73 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
TypeScript
Sterne
8
Forks
2
Ø Merge
1 Std. 34 Min.
Gemergte PRs (30 T.)
6

Beschreibung

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.js anchoredPopover)
    • query tab strip → tablist/tab (src/ui/tabs.js)
  • aria-live on 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:none is 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 test green 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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie mit dem Lesen von src/ui/editor-complete.js, src/ui/file-menu.js, app.js, src/ui/tabs.js, src/ui/toast.js und den aufgeführten outline:none-Regeln in src/styles.css. Führen Sie npm test aus, um das Coverage-Gate pro Datei zu verstehen, und überprüfen Sie anschließend die genannten Abnahmekriterien für Screenreader, Tastaturfokus, aria-live und reduzierte Bewegungen mit dem erwähnten Screenreader-/Tastaturtest.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
css, javascript
Bereich
accessibility, frontend, testing-qa
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.