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

[FEATURE] Ajouter des états succès/erreur au Checkbox tile (PIX-12834) #669

Merged
merged 4 commits into from
Jun 14, 2024

Conversation

yannbertrand
Copy link
Member

@yannbertrand yannbertrand commented Jun 10, 2024

🎄 Problème

Dans la variante tile, on souhaite avoir un design particulier si une checkbox/radio cochée et isDisabled est en succès ou en erreur.

🎁 Proposition

Ajouter un paramètre state qui prend les valeurs suivantes : neutral (défaut), success, error.

Maquettes : https://www.figma.com/design/47MuKB1wXoLEgMal06idN3/R%26D-DevComp?node-id=3835-4921&m=dev

🌟 Remarques

RAS

🎅 Pour tester

Vérifier les nouvelles stories et les différents états mélangés.

@yannbertrand yannbertrand added cross-team Toutes les équipes de dev team-devcomp labels Jun 10, 2024
@yannbertrand yannbertrand self-assigned this Jun 10, 2024
@pix-bot-github
Copy link

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

Base automatically changed from docs-checkbox-radio to dev June 10, 2024 09:37
@yannbertrand yannbertrand marked this pull request as ready for review June 10, 2024 12:10
@yannbertrand yannbertrand force-pushed the pix-12834-add-checkbox-states branch from 2d6ef1a to 352ddaa Compare June 10, 2024 12:11
@reibecca
Copy link
Contributor

Petit soucis au niveau de l'icône quand le state="error"

Capture d’écran 2024-06-10 à 14 19 37

@yannbertrand yannbertrand force-pushed the pix-12834-add-checkbox-states branch 2 times, most recently from d6312d2 to 80a1cc1 Compare June 12, 2024 11:56
@xav-car
Copy link
Contributor

xav-car commented Jun 13, 2024

J'ai l'impression que les icones ne sont pas centré dans leur bulles. ( c'est peut être que moi ) 🤔

@yannbertrand
Copy link
Member Author

J'ai l'impression que les icones ne sont pas centré dans leur bulles. ( c'est peut être que moi ) 🤔

Ouais je suis pas convaincu des svg là ça fait des trucs bizarres. Peut être qu'il y a une autre approche qui serait meilleure ?

@reibecca reibecca force-pushed the pix-12834-add-checkbox-states branch 3 times, most recently from 8aef4a5 to c9f3580 Compare June 13, 2024 15:29
@reibecca reibecca force-pushed the pix-12834-add-checkbox-states branch 2 times, most recently from e70be27 to 1d95ff8 Compare June 14, 2024 14:55
@reibecca reibecca force-pushed the pix-12834-add-checkbox-states branch from 1d95ff8 to c291a35 Compare June 14, 2024 15:01
@pix-service-auto-merge pix-service-auto-merge merged commit 3fa3957 into dev Jun 14, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12834-add-checkbox-states branch June 14, 2024 15:05
pix-service-auto-merge pushed a commit that referenced this pull request Jun 14, 2024
# [46.10.0](v46.9.5...v46.10.0) (2024-06-14)

### 🚀 Amélioration

- [#669](#669) Ajouter des états succès/erreur au Checkbox `tile` (PIX-12834)
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 46.10.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.

7 participants