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

Multiple corrections sur le DSFR #188

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

christophehenry
Copy link
Contributor

@christophehenry christophehenry commented Dec 11, 2024

🎯 Objectif

  • Simplification des templates de form_field_snippets en définissant aria-describedby dans le filtre dsfr_input_class_attr
  • Remplacement de field.label_tag par field.label lorsque <label> ou <legend> sont explicitement utilisés dans le template (ça provoquait l'imbrication d'un <label> dans un <label> ou <legend>)
  • Prise en charge des cases à cocher et des boutons radio inline
  • Ajout de la possibilité de définir des champs inline avec le nouveau filtre dsfr_inline.
  • Réécriture de checkboxselectmultiple_snippet.html et radioselect_snippet.html pour être cohérent avec la documentation DSFR.

@christophehenry
Copy link
Contributor Author

Wait, je croyais que Django 3 était plus supporté ? Comment #168 a pu même passer ?

@christophehenry christophehenry force-pushed the che/multiple-fixes branch 2 times, most recently from db7454e to c5f55bf Compare December 12, 2024 09:04
@Ash-Crow
Copy link
Collaborator

Ash-Crow commented Dec 12, 2024

Le support de Django 3 doit être retiré dans la prochaine version (cf. le README)

@christophehenry
Copy link
Contributor Author

christophehenry commented Dec 12, 2024

Je comprends pas comment les tests ont pu passer. Cette ligne ne fonctionne pas dans Django 3.

Edit : voilà normalement, c'est réparé.

@christophehenry christophehenry force-pushed the che/multiple-fixes branch 2 times, most recently from 35e28d8 to bbfbb24 Compare December 12, 2024 09:36
- Simplify form_field_snippets templates by setting 'aria-describedby' in dsfr_input_class_attr filter
- Don't use 'field.label_tag' when specifying <label> or <legend>
- Add support inline checkboxes and radio buttons
- Add ability to inline fields with new 'dsfr_inline' filter
@Ash-Crow Ash-Crow merged commit 1edd1b8 into numerique-gouv:main Dec 12, 2024
7 of 8 checks 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.

2 participants