Skip to content

Champ de formulaire pour les contrôles segmentés#233

Merged
Ash-Crow merged 5 commits intonumerique-gouv:mainfrom
David-Guillot:feat/segmented-control
Jul 3, 2025
Merged

Champ de formulaire pour les contrôles segmentés#233
Ash-Crow merged 5 commits intonumerique-gouv:mainfrom
David-Guillot:feat/segmented-control

Conversation

@David-Guillot
Copy link
Collaborator

@David-Guillot David-Guillot commented Jun 2, 2025

🎯 Objectif

Implémente le contrôle segmenté en tant que champ de formulaire Django.

Closes #128

🔍 Implémentation

  • Ajout d'un nouveau widget qui gère l'affichage des options ainsi que les différentes variantes (petit, légende inline)
  • S'appuie sur les ExtendedChoices pour permettre l'ajout d'un icône à chaque option

🏕 Amélioration continue

  • Juste le renommage d'une classe dont le nom ne reflétait pas son niveau de généricité
  • Je me suis aussi permis, dans le dernier commit, de mettre à jour la liste des composants pas encore implémentés, qui est maintenant vide 🎉

🖼️ Images

Les trois variantes : classique, légende inline, pas de légende visible (la dernière est également en petite taille)

image

@David-Guillot
Copy link
Collaborator Author

@Ash-Crow je suis en train de percuter que le premier commit (la refacto) contient un breaking change (renommage du paramètre rich_choices en extended_choices). Comment tu voudrais gérer ça ?

  • Éviter ce renommage pas hyper utile
  • Accepter les deux paramètres (en recopiant la valeur de rich_choices dans extended_choices) jusqu'à la prochaine version majeure, avec un warning de déprécation ?

@Ash-Crow
Copy link
Collaborator

Ash-Crow commented Jun 3, 2025

Hello et merci pour cette PR ! (et yay pour le vidage de NOT_YET_IMPLEMENTED_COMPONENTS 🎉)

Je ne suis pas sûr de la meilleure façon de procéder. J'aurais tendance à ne rien toucher mais je suppose que la deuxième solution est la plus pérenne.

@David-Guillot David-Guillot force-pushed the feat/segmented-control branch from 8643c7e to 744cff0 Compare June 4, 2025 12:08
@David-Guillot
Copy link
Collaborator Author

Je ne suis pas sûr de la meilleure façon de procéder. J'aurais tendance à ne rien toucher mais je suppose que la deuxième solution est la plus pérenne.

Je viens d'ajouter un commit de fixup, tu me dis ce que tu en penses 😉

@Ash-Crow Ash-Crow merged commit f8f99a0 into numerique-gouv:main Jul 3, 2025
6 of 7 checks passed
@David-Guillot David-Guillot deleted the feat/segmented-control branch July 8, 2025 10:27
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.

Ajouter les contrôles segmentés dans les formulaires

2 participants