Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ffe-searchable-dropdown-react): fixes isse in talkback #1088

Closed
wants to merge 1 commit into from

Conversation

pethel
Copy link
Contributor

@pethel pethel commented Mar 28, 2021

Screen reader issues på device

Beskrivelse

Team betalning oppdager att denne ikke funker på device(skjermleser) vilket jag bekreftar. Jag var tvungen att fjerne att den opnes på focus før att få det att funke. Fant ikke noen vei run det. Det er manuellt openMenu og closeMenu i blur og focus som ødelegger her.

I vart fall er dom endaste som vil merka en forskjell brukare som bruker tastatur og tabbar seg in i feltet, men og andra sidan har vi ju en pil expandera listan man lett kan trykke på.

edit: Det ser ut som vi kan beholde att den opnes på focus. Problemet ser ut att vare når den får en defaultverdi.

Jag tog også i bruk hooksen før mer kontrol over markupen.

edit2: Ser ut som også defaultverdier løst seg denne snutten så nu tror jag vi har samme funksjionalitet som før men att det virker på talkback med defaultverdier. Vi har forsatt muligens ett problem med Voiceover.

onFocus={e => {
   e.preventDefault();
   onFocus(e);
   if (!selectedItem) {
      openMenu();
   }
}}

Testing

Jag har testat selv på emulator og Talkback men skulle sett pris på hvis noen fler sjekkade ut branchen og gjorde detsamma. Nås på http://10.0.2.4:6060/#!/SearchableDropdown hvis du bruker emulator.

@coveralls
Copy link

coveralls commented Mar 28, 2021

Coverage Status

Coverage increased (+0.03%) to 81.333% when pulling 46a5eed on develop_fix-ally-searchable-dropdown into 55a0a60 on develop.

@pethel pethel force-pushed the develop_fix-ally-searchable-dropdown branch from 706bc18 to 87783b6 Compare March 28, 2021 16:45
@kwltrs kwltrs requested a review from andrea-sb1 March 29, 2021 08:17
@pethel
Copy link
Contributor Author

pethel commented Mar 29, 2021

Sedan har vi denne

downshift-js/downshift#1181

Vi opplever samme sak men det er ikke downshift. Samme problemer opplever vi her https://www.w3.org/TR/wai-aria-practices-1.1/examples/combobox/aria1.1pattern/listbox-combo.html og det skall ju vare fasit egentlig.

@pethel pethel force-pushed the develop_fix-ally-searchable-dropdown branch 5 times, most recently from 16372bf to d72e478 Compare March 30, 2021 07:33
@pethel
Copy link
Contributor Author

pethel commented Apr 8, 2021

Denne funker ikke med voiceover

@pethel pethel deleted the develop_fix-ally-searchable-dropdown branch April 14, 2021 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants