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

[VO-1120] feat(ListItemSkeleton): Add gutters property to mock ListItem behavior #2709

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

cballevre
Copy link
Contributor

No description provided.

Copy link
Collaborator

@JF-Cozy JF-Cozy left a comment

Choose a reason for hiding this comment

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

Selon le code de ce que je comprends on ne change pas le comportement par défaut, mais selon Argos c'est pas super clair pour le ListItemSkeleton vu qu'on a rajouté d'autre cas 🤔 Tu confirmes bien qu'il n'y a pas de BC et que le comportement est le même pour les ListItemSkeleton avec les prop hasSecondary et divider à true/false pour les 2 ? Bref qu'on ne change pas le comportement qu'on avait avant, mais qu'on ne fait QUE rajouter des cas ?

Pour les autres différence Argos, ils ont dû faire un patch de leur côté qui fait que... je vois pas d'autres explications ? 🤷‍♂️

@cballevre
Copy link
Contributor Author

@JF-Cozy non on ne change pas le comportement par défaut on ne fait que rajouter 2 cas (sans marges, avec double marge)sur le Readme. Ce sont les mêmes que pour les ListItem. Si je comprend bien, Argos fait une capture d'écran avec chaque combinaison de cas. Vu qu'il y a 4 cas on doit avoir 16 ListItemSkeleton d'afficher. ça semble correspondre à ce qu'on voit sur les captures d'écran

@cballevre cballevre force-pushed the feat/add-gutters-list-item-skeleton branch from 5b8ae9d to 589a1c4 Compare October 21, 2024 12:50
@cballevre
Copy link
Contributor Author

Capture d’écran 2024-10-21 à 14 51 30

@cballevre cballevre merged commit 01ce10a into master Oct 21, 2024
12 checks passed
@cballevre cballevre deleted the feat/add-gutters-list-item-skeleton branch October 21, 2024 12:59
@cozy-bot
Copy link

🎉 This PR is included in version 111.21.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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants