Régression autocomplete

CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose

https://vitemadose.covidtracker.fr

Other

80 stars 51 forks source link

Régression autocomplete #156

Closed fcamblor closed 3 years ago

fcamblor commented 3 years ago

J'ai détecté 2 régressions sur l'autocomplete :

[x] Une fois que la dropdown est affichée, le filtrage ne se fait plus sur les caractères saisis ensuite (c'était lié au cleanup, lors de d1a847b8f703dad9015c856d14700c907a94044b)
[x] Les départements ne sont plus affichés dans la dropdown (lié au cleanup d1a847b8f703dad9015c856d14700c907a94044b)
[x] Lorsqu'on saisissait un code département, à la fois le département et la première commune étaient en aria-selected=true

Floby commented 3 years ago

Hello et merci @fcamblor pour le fix (et désolé pour la régression que j'ai introduite :disappointed:) Même si c'est une PR plus grosse, je voulais faire remarquer que la PR #148 corrige également ce problème avec l'avantage de fournir 2 suites de tests :

sur le composant en lui-même, y compris la navigation au clavier
sur le service qui gère l'autocomplétion et la récupération des données

Floby commented 3 years ago

Du coup ce que je voulais dire avec mon commentaire précédent, c'est que, sauf urgence MEP, on pourrait peut-être merger préférablement la #148 plutôt que celle-ci.

fcamblor commented 3 years ago

Je préssens un peu qu'on doive MEP aujourd'hui, je préfèrerais merger cette PR afin d'avoir un dev clean.

Sauf si ça te fait des conflits dans tous les sens de rebase #148 par-dessus celle-ci ?

Floby commented 3 years ago

du coup on merge celle-ci, je me demerderai avec les conflits

Floby commented 3 years ago

les conflits avec le harnais de test ça me fait moins peur :)