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] Créer le composant Pix Toast (PIX-9580). #482

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

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Oct 10, 2023

🎄 Problème

Actuellement, nos applications Pix n'ont pas le même design sur les notifications. Si on se réfère au design system, on devrait avoir un composant Pix Toast permettant à nos apps d'être iso (+ avoir des notifs accessible)

Coté Orga : (proche du design system, + travail sur l'a11y) :
Capture d’écran 2023-09-13 à 16 46 34

Coté Admin et Certif : (version de base) :
Capture d’écran 2023-09-13 à 16 54 11

🎁 Proposition

Créer le composant Pix Toast

🌟 Remarques

Pourquoi Toast et pas Notification ? 🤔
Parce que c'est le DSystem qui fait foi et il a dit "Toaster". Mais comme l'a indiqué Clarisse, "Toast" est la bonne formulation.

🎅 Pour tester

Capture d’écran 2024-01-18 à 11 40 25 Capture d’écran 2024-01-18 à 16 21 26 Capture d’écran 2024-01-18 à 16 22 09 Capture d’écran 2024-01-18 à 16 22 22

@AndreiaPena AndreiaPena self-assigned this Oct 10, 2023
@AndreiaPena AndreiaPena changed the title ✨ pix-ui : add pixToaster component [FEATURE] Créer le composant Pix Toaster (PIX-9580). Oct 10, 2023
@pix-bot-github
Copy link

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

@AndreiaPena AndreiaPena force-pushed the pix-9580-add-pix-toaster-component branch from a22e38f to 8cc678a Compare October 10, 2023 14:47
@clarissedmn
Copy link
Contributor

Je pense qu'on dit plutôt un composant Toast plutôt que Toaster :)

@AndreiaPena
Copy link
Member Author

Je pense qu'on dit plutôt un composant Toast plutôt que Toaster :)

J'ai suivi aveuglement le DS mais effectivement, en regardant ailleurs on dit bien Toast :o je change ça et j'en parle aux ui/ux

@AndreiaPena AndreiaPena force-pushed the pix-9580-add-pix-toaster-component branch from 8cc678a to 2bb74cd Compare October 10, 2023 15:26
@AndreiaPena AndreiaPena changed the title [FEATURE] Créer le composant Pix Toaster (PIX-9580). [FEATURE] Créer le composant Pix Toast (PIX-9580). Oct 10, 2023
@AndreiaPena AndreiaPena force-pushed the pix-9580-add-pix-toaster-component branch 5 times, most recently from f1fe268 to 03979ab Compare October 24, 2023 14:11
@AndreiaPena AndreiaPena force-pushed the pix-9580-add-pix-toaster-component branch 10 times, most recently from 793e629 to 5438da1 Compare January 18, 2024 15:15
@AndreiaPena AndreiaPena force-pushed the pix-9580-add-pix-toaster-component branch from 5438da1 to f8d575f Compare January 18, 2024 15:20
@AndreiaPena AndreiaPena added the cross-team Toutes les équipes de dev label Jan 18, 2024
@AndreiaPena AndreiaPena force-pushed the pix-9580-add-pix-toaster-component branch from f8d575f to 0f382a6 Compare January 18, 2024 15:26
@yannbertrand
Copy link
Member

On en fait quoi ?

@AndreiaPena
Copy link
Member Author

AndreiaPena commented Apr 3, 2024

On en fait quoi ?

Il faudrait qu'on le poursuive mais je galère un peu à créer le service. J'aurais besoin d'aide pour le continuer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚧 Development in progress Work In Progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants