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

Korjataan päivän valinta enterillä #5887

Merged
merged 1 commit into from
Nov 8, 2024
Merged

Conversation

tomuli
Copy link
Contributor

@tomuli tomuli commented Oct 29, 2024

Saavutettavuusongelma, enterin käyttö datepickerissä.

Ennen muutosta:

Kun painaa enteriä käytettäessä datepickeriä näppäimistöllä, focus siirtyy datepickerin tekstikenttään ilman että päivä tulee valituksi. Jos yrittää navigoida edellinen/seuraava kuukausi -näppäimillä, focus siirtyy myös tekstikenttään ilman että kuukausi vaihtuu.

Screen.Recording.2024-10-29.at.17.08.50.mov

Muutoksen jälkeen:

Päivän valinta ja kuukausien selaaminen enterillä onnistuu.

datepicker-ScreenRecording2024-11-07.mov

(EDIT) Testeissä on monessa kohtaa käytäntö, jossa datepickerin tekstikenttään "kirjoitetaan" päivämäärä ja painetaan enter. Tämän odotetaan sulkevan datepickerin. Jotta enterillä kuitenkin pystyisi navigoimaan datepickerin kuukausinapeilla sekä valitsemaan halutun päivän, ohitetaan showDatePickerOff jos enter-eventin target tulee muualta kuin tekstikentästä.

@tomuli tomuli added the bug Korjaus olemassa olevaan toiminnallisuuteen label Oct 29, 2024
@tomuli tomuli marked this pull request as draft October 30, 2024 07:06
@tomuli tomuli force-pushed the datepicker-select-with-enter branch from 3d17ac4 to 97afa64 Compare November 7, 2024 11:40
@tomuli tomuli marked this pull request as ready for review November 7, 2024 12:02
@tomuli tomuli merged commit 601bd24 into master Nov 8, 2024
30 checks passed
@tomuli tomuli deleted the datepicker-select-with-enter branch November 8, 2024 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Korjaus olemassa olevaan toiminnallisuuteen
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants