Skip to content

Conversation

@gwennlbh
Copy link
Contributor

@gwennlbh gwennlbh commented Mar 2, 2025

Hello! Déjà merchi bcp pour ce projet c vachement pratique

Quand jsuis arrivée sur la page d'accueil jdois avouer que j'ai direct commencer à remplir mes infos sur la checklist (étant donné que j'ai déjà lu pas mal d'autres ressources sur les procédures administratives avant de trouver ce magnifique site)

Et vu que ya des <select> au deux premieres étapes ça m'a fait étrange que "choisir son prénom" ait pas d'input non plus, dcp voilà une ptite PR

J'ai aussi fait en sorte que ça coche la case des que la value de formData.prénom est truthy (genre pas vide), je l'ai fait dans persistFormData comme ça ça marchera de n'importe où

Aussi je me demande pk ya un form qui utilise nouveauxPrénom comme id de field au lieu de se servir de la valeur globale? ya probablement une raison donc j'y ai pas touché ^^

Le nouveau slot sur StepDetails permet de mettre l'input sous le label pour pas avoir un lien Détails qui traine au milieu de nulle part

Mais vu que dcp l'input a plus de label (pas bien pour l'accessiblilité), j'ai mis "Choisir vos prénoms" dans un span et je rajoute arial-labelledby pour que les liseuses d'écran puisse énoncer à quoi correspond l'input

@gwennlbh gwennlbh force-pushed the set-prenom-at-steps branch from 0b734f2 to e8b6403 Compare March 2, 2025 21:03
...this.formData,
...data
}
if (this.formData.prénom) {
Copy link

Choose a reason for hiding this comment

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

Même si fonctionnel, il serait mieux d'éviter les accents pour des méthodes/variables/tout objet programmatique.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

le truc est que formData définit déjà prénom, c'est pas moi qui l'ait rajouté '^^

(et pis on va être honnête, si en 2025 une techno supporte pas l'utf8 faut arrêter mdr)

à la limite jpeux faire

Suggested change
if (this.formData.prénom) {
if (this.formData["prénom"]) {

mais bon jvois pas vraiment l'intérêt

Copy link
Contributor

Choose a reason for hiding this comment

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

Je suis pour laisser comme ça vu que c'est une variable existante (et je savais pas qu'on avait pas besoin de quote les variables avec un accent, j'ai appris un truc, merci 😎 )

@gwennlbh gwennlbh requested a review from Sutaai March 6, 2025 09:15
Copy link
Contributor

@agateblue agateblue left a comment

Choose a reason for hiding this comment

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

C'est tout bon pour moi, j'ai testé localement et rajouté deux bricoles (un placeholder et un label pour le champ qui n'en avait pas).

je vais merger en mode squash pour enlever le fichier du gitignore, merci pour ton travail !

@agateblue agateblue merged commit a44a40c into Fransgenre:main Mar 6, 2025
1 check passed
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.

3 participants