FilteredActionList: filter input references a missing list while loading or showing a message
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- cypress, react, storybook, typescript
- Ambito
- accessibility, frontend
Direzione di ricerca
Inizia in FilteredActionList e controlla come il campo filtro imposta aria-expanded e aria-controls mentre viene renderizzato bodySpinner, bodySkeleton o un messaggio. Controlla le storie Storybook di caricamento e senza risultati elencate nell’issue, incluso l’uso di SelectPanel. Il lavoro è completo quando gli attributi ARIA del campo corrispondono all’esistenza dell’elenco controllato e axe non segnala errori aria-valid-attr-value in questi stati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
The filter input in FilteredActionList always renders aria-expanded="true" and aria-controls={listId}. When the body shows a loader (loadingType bodySpinner or bodySkeleton) or a message (for example, no results), the list with that id is not rendered, so the combobox points at an element that does not exist.
Assistive technology is told a popup is open when no list is rendered, and axe reports aria-valid-attr-value (critical): Invalid ARIA attribute value: aria-controls=":r0:".
SelectPanel renders FilteredActionList, so it can show the same markup in its loading and empty states.
Steps to reproduce
- Open any of these Storybook stories:
- https://primer.style/react/storybook/?path=/story/components-filteredactionlist-features--loading-with-body-spinner
- https://primer.style/react/storybook/?path=/story/components-filteredactionlist-features--loading-with-body-skeleton
- https://primer.style/react/storybook/?path=/story/components-filteredactionlist-features--with-no-results-message
- Inspect the filter input: it has
aria-expanded="true"andaria-controls=":r0:". document.getElementById(':r0:')returnsnull.- Running axe on the story reports
aria-valid-attr-value.
Version
v38.43.0 (current main)
Browser
All (found with axe-core 4.10 in Chromium)
- Lingua principale
- TypeScript
- Stelle
- 3.9k
- Fork
- 718
- Merge medio
- 2g 15h
- PR unite (30g)
- 66
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
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 primer/react
-
PageHeader.Title does not seem to accept id propForse già presa @TylerJDev l’ha presa 21 giorni fa. Apertacomponent: PageHeader react staff
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
primer/react#8474 · 1 commento · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
The padding prop seems to have no effect with PageLayout.SidebarForse già presa @minwookshin l’ha presa 33 giorni fa. Apertacomponent: PageLayout react staff
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
primer/react#8470 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Spinner "wobbles" on Safari (macOS Tahoe) - also I have fix for itForse già presa @Kavi274 l’ha presa 8 giorni fa. Apertabug component: Spinner react
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
primer/react#8423 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Low contrast in SelectApertawaiting for author response
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
primer/react#8172 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Checkbox: replace useLayoutEffect with ref callback for indeterminate propForse di nuovo libera @akshat-lakhera l’ha presa 13 giorni fa e non c’è nessuna pull request aperta. Apertacomponent: Checkbox react
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
primer/react#7764 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di primer/react
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
ready-for-agent
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
LucasSantana-Dev/Lucky#2951 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
danielmiessler/LifeOS#2567 ·
I maintainer di solito rispondono entro 9 giorni
-
refactor(react-pi): classify base64 images with core's parseBase64Forse già presa @jimmyallinai l’ha presa oggi. Apertamaintenance
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
assistant-ui/assistant-ui#10293 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
brijeshb42/monaco-vim#175 ·