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

Pc 33527 checkbox group with border #15709

Merged
merged 9 commits into from
Jan 13, 2025

Conversation

gmeigniez-pass
Copy link
Contributor

@gmeigniez-pass gmeigniez-pass commented Jan 3, 2025

But de la pull request

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-33527

Objectif
Comme pour les radio group, avoir le variant des checkbox avec bordure, et potentiellement inline, et pouvoir afficher du contenu sous la checkbox quand elle est cochée.
C'est dans le cadre de l'amélioration des offres vitrines côté collectif. Les différents écrans qu'on doit pouvoir développer à terme sont ici, cependant, cette PR ne concerne que le travail sur les checkbox elles-mêmes et pas l'implémentation.

Capture d’écran 2025-01-03 à 15 49 44 Capture d’écran 2025-01-03 à 15 49 50 Capture d’écran 2025-01-03 à 15 49 56 Capture d’écran 2025-01-03 à 15 50 04 Capture d’écran 2025-01-03 à 15 53 56

Vérifications

  • J'ai écrit les tests nécessaires
  • J'ai mis à jour le fichier des plans de tests du portail pro si nécessaire
  • J'ai mis à jour la liste des routes et des titres de pages du portail pro si j'en ai rajouté/modifié ou supprimé une.
  • J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
  • J'ai ajouté des screenshots pour d'éventuels changements graphiques
  • J'ai fait la revue fonctionnelle de mon ticket

Copy link
Contributor

github-actions bot commented Jan 3, 2025

Visit the preview URL for this PR (updated for commit 806d693):

https://pc-pro-testing--pr15709-pc-33527-checkbox-gr-btyipcps.web.app

(expires Wed, 15 Jan 2025 10:31:46 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1

@gmeigniez-pass gmeigniez-pass force-pushed the pc-33527-checkbox-group-with-border branch from 681b6f7 to f527c70 Compare January 3, 2025 16:28
@gmeigniez-pass gmeigniez-pass force-pushed the pc-33527-checkbox-group-with-border branch from f527c70 to bcdfebd Compare January 7, 2025 09:56
@ahello-pass ahello-pass force-pushed the pc-33527-checkbox-group-with-border branch from bcdfebd to 259f128 Compare January 10, 2025 14:20
@ahello-pass ahello-pass force-pushed the pc-33527-checkbox-group-with-border branch from 259f128 to e8d808a Compare January 13, 2025 10:10
@ahello-pass ahello-pass force-pushed the pc-33527-checkbox-group-with-border branch from e8d808a to 806d693 Compare January 13, 2025 10:14
@ahello-pass ahello-pass merged commit 3d249a7 into master Jan 13, 2025
22 checks passed
@ahello-pass ahello-pass deleted the pc-33527-checkbox-group-with-border branch January 13, 2025 14:09
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.

4 participants