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] Organiser les composants comme dans Figma (PIX-14598) #762

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

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Nov 14, 2024

🎄 Problème

Les catégories storybook des composants ne suivent pas la catégorisation définie dans le design système.

🎁 Proposition

Placer les composants dans les bonnes catégories (organisation définie par le design, voir le ticket PIX-14598 associé)

🌟 Remarques

  1. La documentation a été regroupée sous une catégorie Pix UI
  2. Correction de bugs dans le storybook lié à la lib @formatjs/intl

🎅 Pour tester

Voir le storybook https://ui-pr762.review.pix.fr

@bpetetot bpetetot self-assigned this Nov 14, 2024
@pix-bot-github
Copy link

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

@bpetetot bpetetot force-pushed the pix-14598-organize-components-by-figma-categories branch from 1710ec3 to 0b83c95 Compare November 14, 2024 13:18
@bpetetot bpetetot marked this pull request as ready for review November 14, 2024 13:19
@AndreiaPena
Copy link
Member

Le toggle ne devrait pas se trouver dans navigation ?

Capture d’écran 2024-11-14 à 14 33 41 Capture d’écran 2024-11-14 à 14 34 12

@bpetetot
Copy link
Contributor Author

Le toggle ne devrait pas se trouver dans navigation ?

En effet, bien vu ! Je m'étais basé sur le fichier de correspondance (voir dans le ticket) et j'ai lu la mauvaise ligne (j'ai confondu avec le Switch qui est dans form)

@AndreiaPena
Copy link
Member

AndreiaPena commented Nov 14, 2024

Je ne vois pas le nouveau composant PixNavigation dans la liste

celui-ci (screen de dev)
Capture d’écran 2024-11-14 à 14 41 01

@bpetetot bpetetot force-pushed the pix-14598-organize-components-by-figma-categories branch 2 times, most recently from fd457bb to fcbc4ff Compare November 14, 2024 14:38
@bpetetot
Copy link
Contributor Author

@AndreiaPena je n'étais pas à jour avec dev. C'est bon maintenant.

J'ai mis PixNavigation dans Navigation, ce qui me semblait le plus logique... (cc @pierrepougetpix)

@bpetetot bpetetot force-pushed the pix-14598-organize-components-by-figma-categories branch from fcbc4ff to 5d7cab0 Compare November 14, 2024 16:33
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.

5 participants