search

codegouvfr / react-dsfr

🇫🇷 The Frech Government Design system React toolkit
https://react-dsfr.codegouv.studio
MIT License
403 stars 49 forks source link

SelectNext: régression placeholder toujours sélectionnable #256

Closed totakoko closed 3 months ago

totakoko commented 3 months ago

:wave: Je ne connais pas le but recherché de ce commit, mais maintenant, tous les les attributs d'une Option ne sont pas passés au DOM (disabled et selected), notamment pour le placeholder qui est toujours sélectionnable.

Comportement en 1.9.3 : on ne peut pas sélectionner le placeholder image

Comportement en 1.9.9 : on peut sélectionner le placeholder image

Par ailleurs, côté DSFR natif, je vois qu'ils permettent d'utiliser un attribut hidden sur le placeholder pour qu'il n'apparaisse pas dans la liste. Est-ce que ça serait facile à implémenter ? J'imagine que ça devrait être un paramètre supplémentaire facultatif au composant.

garronej commented 3 months ago

@ddecrulle Tu peut regarder?
Je savais bien qu'il y avais une subtilité dont je ne me rappelais plus

@totakoko Merci pour l'issue