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

[TECH] Affiche toutes les icônes dans la story de PixIcon #738

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

lionelB
Copy link
Member

@lionelB lionelB commented Oct 8, 2024

🎄 Problème

C'est pénible de ne pas avoir la liste de toutes les icônes disponibles dans la story du composant.

🎁 Proposition

On affiche toutess les icônes dispo

🌟 Remarques

J'aime travailler avec storybook
image

🎅 Pour tester

Afficher la story de PixIcon

@lionelB lionelB added the 🚧 Development in progress Work In Progress label Oct 8, 2024
@pix-bot-github
Copy link

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

@lionelB lionelB force-pushed the add-all-icon-pix-icon-story branch from bb4afd3 to af51562 Compare October 8, 2024 14:16
@lionelB lionelB changed the title feat(icons): add all icons in story [TECH] Affiche tout les icons dans la story de PixIcons Oct 8, 2024
@lionelB lionelB force-pushed the add-all-icon-pix-icon-story branch 2 times, most recently from 33791e1 to fb9eea0 Compare October 8, 2024 14:59
@lionelB lionelB self-assigned this Oct 8, 2024
@Jeyffrey Jeyffrey changed the title [TECH] Affiche tout les icons dans la story de PixIcons [TECH] Affiche toutes les icônes dans la story de PixIcons Oct 8, 2024
@Jeyffrey Jeyffrey changed the title [TECH] Affiche toutes les icônes dans la story de PixIcons [TECH] Affiche toutes les icônes dans la story de PixIcon Oct 8, 2024
@lionelB lionelB force-pushed the add-all-icon-pix-icon-story branch 2 times, most recently from 1b1d6d6 to cde748c Compare October 9, 2024 07:40

## Usage

```html
<PixIcon @name="powerSettings", @plainIcon={{true}} />
Copy link
Contributor

Choose a reason for hiding this comment

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

Y a une virgule en trop entre les 2 paramètres

@AndreiaPena
Copy link
Member

Cool d'avoir la liste :)

Possible d'agrandir la largeur ? J'ai les icônes tout à droite coupés

Capture d’écran 2024-10-09 à 12 02 41

@lionelB lionelB force-pushed the add-all-icon-pix-icon-story branch from cde748c to 435c183 Compare October 9, 2024 11:50
@lionelB lionelB force-pushed the add-all-icon-pix-icon-story branch from 435c183 to 87febfc Compare October 9, 2024 12:10
@lionelB
Copy link
Member Author

lionelB commented Oct 9, 2024

@AndreiaPena c'est corrigé, merci !

@pix-service-auto-merge pix-service-auto-merge merged commit 1b0d1cc into dev Oct 9, 2024
11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the add-all-icon-pix-icon-story branch October 9, 2024 15:51
pix-service-auto-merge pushed a commit that referenced this pull request Oct 9, 2024
## [46.15.1](v46.15.0...v46.15.1) (2024-10-09)

### 🏗️ Tech

- [#738](#738) Affiche toutes les icônes dans la story de PixIcon
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 46.15.1 🎉

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.

5 participants