Skip to content

✨(ats-presentation) add base form components#682

Merged
ShallowRed merged 5 commits into
mainfrom
add-form-components
Jun 5, 2026
Merged

✨(ats-presentation) add base form components#682
ShallowRed merged 5 commits into
mainfrom
add-form-components

Conversation

@ShallowRed
Copy link
Copy Markdown
Collaborator

@ShallowRed ShallowRed commented Jun 4, 2026

📝 Description

🎸 Ajouts de composants de formulaire basiques :

  • input
  • textarea
  • checkbox et checkbox group
  • radio et radio group
  • switch

🏷️ Type de changement

  • 🐛 Correction de bug
  • 🎢 Nouvelle fonctionnalité (changement non bloquant qui ajoute une fonctionnalité)
  • 🥁 Changement breaking (modification ou fonctionnalité qui pourrait casser le fonctionnement existant) nécessitant une mise à jour de la documentation
  • 📚 Mise à jour de la documentation
  • ♻️ Refactorisation
  • 🔧 Changement technique

✅ Liste de contrôle

  • 💅 J’ai ajouté ou mis à jour les tests appropriés.
  • 📝 J’ai mis à jour ou ajouté la documentation nécessaire.
  • 🚀 J’ai pris en compte l’impact sur les performances, la sécurité et l’expérience utilisateur.
  • 👀 J’ai demandé une revue à une personne de l’équipe.

@ShallowRed ShallowRed linked an issue Jun 4, 2026 that may be closed by this pull request
4 tasks
@ShallowRed ShallowRed added added PR: backward compatible addition storybook labels Jun 4, 2026
@ShallowRed ShallowRed self-assigned this Jun 4, 2026
@ShallowRed ShallowRed requested a review from smdsgn June 4, 2026 16:19
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

📖 Storybook de prévisualisation : https://betagouv.github.io/csplab/storybook/pr-682/

@ShallowRed ShallowRed requested a review from AliceGourbat June 4, 2026 16:20
@ShallowRed ShallowRed marked this pull request as ready for review June 4, 2026 16:20
@ShallowRed ShallowRed force-pushed the add-form-components branch from 967d38a to 76ab1f6 Compare June 4, 2026 16:22
Copy link
Copy Markdown
Collaborator

@smdsgn smdsgn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Les stories qui lazy-loadent le chunk _plugin-vue_export-helper-*.js cassent. Ce sont celles avec du style scoped j'imagine. Le problème sembel venir de Jekyll. Il faut sans doute ajouter un fichier .nojekyll dans le dossier Storybook avant le deploy.

@smdsgn
Copy link
Copy Markdown
Collaborator

smdsgn commented Jun 5, 2026

@AliceGourbat C'est corrigé, tu peux faire la recette si tu veux.

@ShallowRed ShallowRed force-pushed the add-form-components branch from 15a54fc to 87523e3 Compare June 5, 2026 08:16
@AliceGourbat
Copy link
Copy Markdown
Collaborator

AliceGourbat commented Jun 5, 2026

  • Chech box : je ne suis pas sûr de comprendre les cas d'usage de l'état indeterminé
  • Input :
    • Disabled : j'ai peur que le contraste ne soit pas accessible
    • password : pourrait-on ajouter un oeil pour afficher le mdp ?
  • radio : manque le cas erreur
  • Switch : j'ai un doute sur l'emplacement du libellé notamment quand on voit l'impact de l'erreur

Copy link
Copy Markdown
Collaborator

@AliceGourbat AliceGourbat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ShallowRed
Copy link
Copy Markdown
Collaborator Author

ShallowRed commented Jun 5, 2026

Chech box : je ne suis pas sûr de comprendre les cas d'usage de l'état indeterminé

C'était dans l'ui kit, je me suis dit que c'était peut-être intéressant (pour des cas de sélection groupée ou de hierarchie), mais pas sur !
Capture d’écran 2026-06-05 à 11 57 21


Disabled : j'ai peur que le contraste ne soit pas accessible

C'est vrai que c'est pas idéal mais ça reproduit ce qui se fait côté dsfr, je pense qu'il y aura peu de cas d'usages avec champ visible + une valeur + disabled.

Capture d’écran 2026-06-05 à 11 51 38

password : pourrait-on ajouter un oeil pour afficher le mdp ?

Ce sera plutôt un composant parent qui composera le champ mdp seul + un bouton avec œil voir autre chose


radio : manque le cas erreur

Le radio ne sera jamais utilisé seul (ça crééerait même une erreur dans le code), mais toujours dans un radiogroup qui a un cas d'erreur. Dans les faits c'est pas très intéressant d'avoir mis une story radio seule dédiée, j'aurais pu en faire qu'une sur le radiogroup.


Switch : j'ai un doute sur l'emplacement du libellé, notamment quand on voit l'impact de l'erreur

Je suis pas sur d'avoir compris mais celui là reste à pas mal certes à pas mal étoffer pour se rapprocher du dsfr.

@ShallowRed ShallowRed merged commit ee3b2aa into main Jun 5, 2026
28 checks passed
@ShallowRed ShallowRed deleted the add-form-components branch June 5, 2026 10:03
github-actions Bot added a commit that referenced this pull request Jun 5, 2026
@github-actions github-actions Bot mentioned this pull request Jun 5, 2026
@AliceGourbat
Copy link
Copy Markdown
Collaborator

  • check box : Ah parfait c'était le cas auquel je pensais mais c'est le nom "indéterminé" qui m'a perdu 👍
  • field Disabled : c'était plus dans l'idée de garder le même rendu mais peux-être modifier les teintes de gris pour garder le contraste ok. Mais si c'est trop complexe, no worries on se cale au DSFR
  • password : ok 👍
  • radio : oui tu as raison 👍
  • switch : globalement si on se rapproche de celui du DSFR ça me va. C'était plutôt le fait d'avoir le libellé à droite avec le libellé en rouge en cas d'erreur que je trouvais problématique. Mais si on le gère comme radio & check (comme dans le dsfr) c'est niquel ! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added PR: backward compatible addition storybook

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mise en place du kit UI — Reka UI + couche style DSFR

3 participants