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

feat(redmine 1246926): hasOnlyOneOpenMenu option for SidebarMenu, added utilities #13

Merged
merged 4 commits into from
Sep 25, 2023

Conversation

QuentinLeCaignec
Copy link
Contributor

refactor: reworked CollapseButton into Uncontrolled and Controlled versions

refactor: modified CollapseButton stories, added expanded detail to stories (desc and default columns)

@changeset-bot
Copy link

changeset-bot bot commented Sep 21, 2023

🦋 Changeset detected

Latest commit: 1e094d2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@smile/react-front-kit Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@QuentinLeCaignec QuentinLeCaignec force-pushed the feat/arborescence_folder branch 2 times, most recently from bfb467c to 8af25cf Compare September 22, 2023 12:40
…ed utilities

refactor: reworked CollapseButton into Uncontrolled and Controlled versions

refactor: modified CollapseButton stories, added expanded detail to stories (desc and default columns)
@QuentinLeCaignec
Copy link
Contributor Author

QuentinLeCaignec commented Sep 25, 2023

@tonai J'ai ajusté selon tes commentaires. Je suis aussi passé sur les components existants pour rajouter la ligne /* decscription du component **/, je pense qu'on peut prendre le réflexe de la mettre même si elle est vide par réflexe.

Par contre problème, pour le cas spécifique de la seconde story de SidebarMenu (nommée OnlyOneOpenMenu), les commentaires /* **/ au dessus de la prop de CollapseButtonControlled n'est pas reconnu par storybook pour une raison mystérieuse. Ça marche ailleurs, mais pas ici. Pourtant le meta de cette story importe bien CollapseButtonControlled et pas CollapseButton, je ne comprend pas.

J'ai linté le tout est ça a modifié légèrement Table avec des sauts de lignes et une simplification des imports, je pense pas que ça devrais créer de conflits avec @vapersmile plus tard mais si il faut j'enlève de cette branche.

@tonai
Copy link
Contributor

tonai commented Sep 25, 2023

Par contre problème, pour le cas spécifique de la seconde story de SidebarMenu (nommée OnlyOneOpenMenu), les commentaires /* **/ au dessus de la prop de CollapseButtonControlled n'est pas reconnu par storybook pour une raison mystérieuse. Ça marche ailleurs, mais pas ici. Pourtant le meta de cette story importe bien CollapseButtonControlled et pas CollapseButton, je ne comprend pas.

Peux-tu tester si tu as le même problème en créant 2 fichiers .stories (un pour chaque composant, et donc une doc pour chaque composant)

@QuentinLeCaignec
Copy link
Contributor Author

QuentinLeCaignec commented Sep 25, 2023

Par contre problème, pour le cas spécifique de la seconde story de SidebarMenu (nommée OnlyOneOpenMenu), les commentaires /* **/ au dessus de la prop de CollapseButtonControlled n'est pas reconnu par storybook pour une raison mystérieuse. Ça marche ailleurs, mais pas ici. Pourtant le meta de cette story importe bien CollapseButtonControlled et pas CollapseButton, je ne comprend pas.

Peux-tu tester si tu as le même problème en créant 2 fichiers .stories (un pour chaque composant, et donc une doc pour chaque composant)

Yep, ça fonctionne comme ça. Tant que l'autre fichier est du même nom que la story qu'il contient (ici le fichier et sa story son Controlled, et que j'importe le title avec les paramètres du meta de CollapseButton, storybook le range dans le même dossier et ça rend bien.
Ça marche aussi si le nom du fichier et de sa story sont autre choses bien sûr mais ça crée des sous-dossiers moins propres je trouve.

Non seulement maintenant la description marche mais ça détecte correctement les Actions aussi.

@QuentinLeCaignec QuentinLeCaignec merged commit b470faf into 0.2.x Sep 25, 2023
1 check passed
@QuentinLeCaignec QuentinLeCaignec deleted the feat/arborescence_folder branch September 25, 2023 13:54
tonai pushed a commit that referenced this pull request Oct 4, 2023
…ed utilities (#13)

* feat(redmine 1246926): hasOnlyOneOpenMenu option for SidebarMenu, added utilities

refactor: reworked CollapseButton into Uncontrolled and Controlled versions

refactor: modified CollapseButton stories, added expanded detail to stories (desc and default columns)
@tonai tonai mentioned this pull request Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants