|Slash] Suggestion - nouveau composant interne Fieldset#1375
|Slash] Suggestion - nouveau composant interne Fieldset#1375arnaudforaison wants to merge 2 commits intomainfrom
Conversation
arnaudforaison
requested review from
GuillaumeKESTEMAN,
JLou,
MartinWeb and
pplancq
| <div className={classNameContainerLabel}> | ||
| <legend |
There was a problem hiding this comment.
l'élément legend doit être le premier enfant d'un fieldset et ne peut contenir que des élément de phrasé, donc pas de div en enfant de legend
https://developer.mozilla.org/fr/docs/Web/HTML/Reference/Elements/legend
https://developer.mozilla.org/fr/docs/Web/HTML/Guides/Content_categories#contenu_phrasé
| aria-required={required} | ||
| aria-invalid={isInvalid} |
There was a problem hiding this comment.
aria-required et aria-invalid ne peuvent être placé sur le fieldset que si celui co à le role radiogroup
d'ailleurs en cas de radiogroup il aussi ajout le aria-errormessage et le aria-describeBy qui ne doit pas être sur le élément input radio
contrairement pour le group (dédier au checkbox) la il faut passer les arria au input checkbox
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Reference/Attributes/aria-required#associated_roles
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Reference/Attributes/aria-errormessage#associated_roles
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Reference/Attributes/aria-invalid#associated_roles
| groupClassName, | ||
| { | ||
| "af-form__group--required": actualRequired, | ||
| "af-form__group--label-top": labelPosition === "top", |
There was a problem hiding this comment.
comment tu arrive à styliser le fieldset et le legend pour que le legend ce place à gauche et les input à droite ? car il me semble que ce n'est pas possible pour le moment ?
| disabled, | ||
| ...otherProps, | ||
| })} | ||
| {forceDisplayMessage ? ( |
There was a problem hiding this comment.
pourquoi ce forceDisplayMessage ?
si on a un message on affiche le FieldError sinon le HelpMessage
There was a problem hiding this comment.
Renommer le fichier en Fieldset.tsx
pplancq
added
theme-distributeur
3 participants
Il existe le composant
Fieldqui permet d'avoir tous les inputs standardisés et mutualisés en terme de gestion d'erreur, des messages ,className, etc ...Par contre ce composant
Fieldest adapté uniquement pour les "single" input, un input number ou text ou textarea.Si on veut faire des groupements ma suggestion est le composant
Fieldsetqui permet une meilleur gestion des "multi" inputs notamment en terme d'accessibilité.le commun entre
FieldetFieldseta été sorti dans des hooks.