Feature/arc 3788 search block - #368
Conversation
| }) | ||
| ); | ||
|
|
||
| console.log('result', result); |
There was a problem hiding this comment.
console.log('result', result) staat er nog in.
| query: searchTerm | ||
| ? { | ||
| zoekterm: searchTerm, | ||
| format: mediaType, | ||
| } | ||
| : {}, |
There was a problem hiding this comment.
Bij een lege searchTerm wordt de query {}, dus de gekozen format verdwijnt mee. Kies "Alle video", klik zoeken zonder iets te typen, en je komt op /zoeken zonder filter.
De FA koppelt dat niet aan een zoekterm: "Als een optie (andere dan 'Zoek alles') hier geselecteerd is en je start een zoekactie, dan wordt deze zoekactie uitgevoerd met de gekozen filter voor dit objecttype." Ik zou format altijd meegeven, of minstens wanneer het niet all is.
|
|
||
| const actionProps = { | ||
| tabIndex: 0, | ||
| role: 'button', |
There was a problem hiding this comment.
De FA vermeldt letterlijk: "Escape = sluit 'lagen bovenop de webpagina, e.g. dropdowns, blade, modals, etc." Nu gaat hij enkel dicht via de trigger of door een optie te kiezen. Klikken buiten de dropdown doet ook niets.
Verder luistert onKey(evt, [...keysEnter], …) alleen op Enter. Op een element met role="button" verwacht een screenreader-gebruiker ook Space, en de FA noemt die apart.
bf9e4e9 to
ac5f1fc
Compare
2 participants
No description provided.