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

Correction du style des checkbox à sélection multiple #205

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

David-Guillot
Copy link

🎯 Objectif

Corrige un bug d'affichage des checkboxes groupées via le widget CheckboxSelectMultiple.

🔍 Implémentation

Juste ajouter la classe fr-checkbox-group au fieldset entourant les checkboxes.

🖼️ Images

Avant :

image

Après :

image

Add missing `fr-checkbox` class to the fieldset
@David-Guillot
Copy link
Author

Note : j'ai mis la version à 2.1.2 sur cette PR, donc il faut merger #204 avant pour que ça se suive 😬

@Ash-Crow
Copy link
Collaborator

Ash-Crow commented Feb 4, 2025

Merci !

Note : j'ai mis la version à 2.1.2 sur cette PR, donc il faut merger #204 avant pour que ça se suive 😬.

On peut publier les deux dans la même release :) (2.1.1 donc)

De toute façon, je n'aurai pas le temps de regarder avant le prochain sprint la semaine prochaine

@David-Guillot
Copy link
Author

Ah OK, force-pushé à 2.1.1 donc 😅 . Pas de souci pour le délai de l'intégration de ces petites modifs, aucune urgence de mon côté 👍

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