Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configuration des classes CSS dans le template en même temps que dans le Form #101

Merged

Conversation

christophehenry
Copy link
Contributor

@christophehenry christophehenry commented Jan 30, 2024

🎯 Objectif

dsfr_form_field est inutile s'il n'est pas utilisé avec DsfrBaseForm puisque la classe CSS est paramétrée dans DsfrBaseForm. C'est la seule chose qui est configurée au niveau du formulaire et pas au niveau template, ce qui est contre-intuitif.

Cette PR ajoute cette logique dans un filtre à utilisé sur chaque field.

@christophehenry christophehenry force-pushed the che/fix-dsfr_form_field branch 2 times, most recently from 9ec7e3c to 8b5abf0 Compare January 31, 2024 13:29
@christophehenry christophehenry changed the title Configuration des classes CSS dans le template plutôt que dans le Form Configuration des classes CSS dans le template en même temps que dans le Form Jan 31, 2024
@christophehenry christophehenry force-pushed the che/fix-dsfr_form_field branch from 8b5abf0 to b798075 Compare February 5, 2024 15:15
@christophehenry
Copy link
Contributor Author

Rebased.

@Ash-Crow Ash-Crow merged commit 7258eae into numerique-gouv:main Feb 22, 2024
7 of 8 checks passed
@Ash-Crow
Copy link
Collaborator

Merci !

@christophehenry christophehenry deleted the che/fix-dsfr_form_field branch March 5, 2024 14:41
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.

2 participants