InfoTooltip is not keyboard reachable and has no accessible name
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 84/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- react, typescript
- Ambito
- accessibility, frontend
Direzione di ricerca
Inizia da src/custom/CustomTooltip/infoTooltip.tsx e segui il modo in cui InfoTooltip passa il titolo e le props a CustomTooltip. Verifica che il controllo risultante sia raggiungibile da tastiera, abbia un nome accessibile e continui ad aprire il testo della guida quando riceve il focus, preservando i siti di chiamata esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
InfoTooltip renders its icon in a plain <div>:
https://github.com/layer5io/sistent/blob/master/src/custom/CustomTooltip/infoTooltip.tsx
<CustomTooltip title={helpText} {...props}>
<div style={{ display: 'flex', ... }}>
<InfoOutlinedIcon {...iconSmall} />
</div>
</CustomTooltip>
That element has no accessible name, no role, and no tab index, so the help
text it carries is unreachable for a keyboard or screen-reader user:
- nothing announces it in browse mode - the icon is decorative markup;
- it cannot be focused, and MUI opens a tooltip on focus, so the text has no
keyboard route at all; CustomTooltiphands MUI a ReactNode title, and MUI'sTooltiponly sets
aria-labelfrom a title when that title is a string (titleIsString ? title : null), so the closed tooltip contributes no name either.
Why it matters here
meshery-cloud renders a per-field information affordance on every schema-driven
form (the field's description from meshery/schemas). InfoTooltip is
exactly the component for that job, but it cannot be used: our accessibility
requirement is that the affordance is reachable and announced. We hand-assemble
CustomTooltip + IconButton component="span" + an aria-label instead, which
is duplicated design-system markup living in a consumer.
Suggested fix
Render the icon in a ButtonBase/IconButton (component="span" keeps it
legal inside a <label>-adjacent layout) rather than a <div>, and accept an
aria-label - defaulting to the helpText when it is a string - so the
component names itself. That keeps every existing call site working and makes
the affordance reachable by Tab, which also opens the tooltip.
Happy to raise the PR if the shape above looks right.
- Lingua principale
- TypeScript
- Stelle
- 137
- Fork
- 239
- Merge medio
- 4g 14h
- PR unite (30g)
- 5
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di layer5io/sistent
-
component/ui help wanted kind/enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
layer5io/sistent#1772 · 4 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
layer5io/sistent#1749 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
ESM build is not loadable by Node: extensionless 'lodash/debounce' specifier in dist/index.mjsAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
layer5io/sistent#1739 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: Catalog design card titles truncate to single line with no way to see full nameForse già presa @Maanvi212006 l’ha presa 4 giorni fa. Apertakind/bug
layer5io/sistent#1849 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
area/ui component/components kind/enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di layer5io/sistent
Issue simili
-
module-request
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
ports get and web print 'Port N already in use, trying next...' for every busy port they skipAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
appandflow/stim#1604 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
lingdojo/kana-dojo#31060 · 1 commento · 5 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
SSH workspace restore rewrites relative symlinks into the deleted sync-back staging directoryAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
paperclipai/paperclip#14173 ·
I maintainer di solito rispondono entro 1 giorno
-
needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno