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

[BUGIFX] Ne plus passer un un wrapped label pour les checkbox/radiobutton (PIX-12562) #644

Merged
merged 2 commits into from
May 17, 2024

Conversation

xav-car
Copy link
Contributor

@xav-car xav-car commented May 16, 2024

🎄 Problème

le wrapped element pose régulièrement souci ( notamment dans le cas d'un label en screenreader only sur la page des epreuves )

🎁 Proposition

ne plus passer par un gap pour gérer l'espacement mais un padding. qui permettra le click dans la zone blanche. et nous évitera tout un tas de bidouille pour que tout fonctionne partout

🌟 Remarques

RAS

🎅 Pour tester

Vérifier que tout fonctionne sur les épreuves

@xav-car xav-car self-assigned this May 16, 2024
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr644.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr644/environment

@xav-car xav-car added the 🚧 Development in progress Work In Progress label May 16, 2024
@xav-car xav-car changed the title Remove wrapped label [FIX] Ne plus passer un un wrapped label pour les checkbox/radiobutton (PIX-XXXX) May 16, 2024
@xav-car xav-car changed the title [FIX] Ne plus passer un un wrapped label pour les checkbox/radiobutton (PIX-XXXX) [FIX] Ne plus passer un un wrapped label pour les checkbox/radiobutton (PIX-12562) May 17, 2024
@xav-car xav-car changed the title [FIX] Ne plus passer un un wrapped label pour les checkbox/radiobutton (PIX-12562) [BUGIFX] Ne plus passer un un wrapped label pour les checkbox/radiobutton (PIX-12562) May 17, 2024
@xav-car xav-car marked this pull request as ready for review May 17, 2024 07:33
@La-toile-cosmique
Copy link
Contributor

@xav-car
Copy link
Contributor Author

xav-car commented May 17, 2024

ça semble poser problème sur les autres QCU/QCM

https://app-pr8950.review.pix.fr/challenges/recT0Ks2EDgoDgEKc/preview https://app-pr8950.review.pix.fr/challenges/recLt9uwa2dR3IYpi/preview

une correction a été effectué dessus. normalement le rebuild devrait corriger ce qu'il faut.

@xav-car
Copy link
Contributor Author

xav-car commented May 17, 2024

ça semble poser problème sur les autres QCU/QCM

https://app-pr8950.review.pix.fr/challenges/recT0Ks2EDgoDgEKc/preview https://app-pr8950.review.pix.fr/challenges/recLt9uwa2dR3IYpi/preview

https://ui-pr644.review.pix.fr/?path=/story/form-checkbox--default , sur PixUI c'est OK. Scalingo galère à déployer.

Copy link
Member

@yannbertrand yannbertrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤞 ça m'a l'air bon dans nos exemples Storybook

@pix-service-auto-merge pix-service-auto-merge merged commit 4b671ce into dev May 17, 2024
3 of 4 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the remove-wrapped-label branch May 17, 2024 14:35
pix-service-auto-merge pushed a commit that referenced this pull request May 17, 2024
# [46.2.0](v46.1.0...v46.2.0) (2024-05-17)

### 🚀 Amélioration

- [#642](#642) Mise à jour des tailles des boutons (PIX-12418)

### ☕ Autre

- [#644](#644) Ne plus passer un un wrapped label pour les checkbox/radiobutton (PIX-12562)
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 46.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants