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.

Sign up for GitHub

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

Jump to bottom

[FEATURE] Organiser les composants comme dans Figma (PIX-14598) #762

Merged
merged 5 commits into from
Nov 18, 2024
Merged

[FEATURE] Organiser les composants comme dans Figma (PIX-14598) #762

merged 5 commits into from
Nov 18, 2024

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Nov 14, 2024 •
edited
Loading

🎄 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 •
edited
Loading

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
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14598-organize-components-by-figma-categories branch from 5d7cab0 to 499d3c6 Compare November 18, 2024 13:52
@pix-service-auto-merge pix-service-auto-merge merged commit 2a4e929 into dev Nov 18, 2024
5 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14598-organize-components-by-figma-categories branch November 18, 2024 13:56
pix-service-auto-merge pushed a commit that referenced this pull request Nov 18, 2024
@semantic-release-bot
chore(release): 48.6.0 [skip ci]
c4cbdaf 
# [48.6.0](v48.5.0...v48.6.0) (2024-11-18)

### 🚀 Amélioration

- [#762](#762) Organiser les composants comme dans Figma (PIX-14598)
@pix-service-auto-merge
Copy link
Contributor

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

@mcampourcy mcampourcy mcampourcy approved these changes

@er-lim er-lim er-lim approved these changes

@VincentHardouin VincentHardouin VincentHardouin approved these changes

@xav-car xav-car xav-car approved these changes

Assignees

@bpetetot bpetetot

Labels
cross-team Toutes les équipes de dev Design review OK released 🚀 Ready to Merge team-acces Tech review OK
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@bpetetot @pix-bot-github @AndreiaPena @pix-service-auto-merge @mcampourcy @er-lim @VincentHardouin @xav-car @pierrepougetpix