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(Tabs)!: add default selected tab option #885

Merged
merged 5 commits into from
Jun 7, 2024
Merged

Conversation

hebernardEquisoft
Copy link
Contributor

@hebernardEquisoft hebernardEquisoft commented May 27, 2024

DS-1155

Description

Ici on vient ajouter l'option d'avoir un tab selectionné par défaut.

Tests fonctionnels

  • Valider le bon fonctionnement de tab selectionné par défaut
  • Valider le bon fonctionnement des tabs et qu'il n'y a aucune régression

@hebernardEquisoft hebernardEquisoft requested a review from a team as a code owner May 27, 2024 18:51
Copy link

Storybook for this build: https://ds.equisoft.io/pr-885/

Copy link

Webapp for this build: https://ds.equisoft.io/pr-885/webapp/

@savutsang
Copy link
Contributor

savutsang commented May 28, 2024

J'ai de la misere un peu que ca soit dans les items, ca fait plus de sens que ca soit un prop de Tabs a place.

Ca va de pair avec le controlled "selected" (lorsqu'on ajoutera ca) et c'est certain que pour lui faut le mettre au niveau du Tabs, car c'est un truc qui se switch souvent et c'est plus pratique a l'utiliser lorsque c'est pas embedded dans les data.

C'est le format qu'on voit dans les autre DS, et aussi ce qu'on a pour DropdownList.

Pour l'argument que ca marche pas car le id n'est pas obligatoire ici, je dirais: si tu veux pouvoir selected ou defaultSelected un tab, met un id a ce tab. P-t meme qu'on devrait rendre les id obligatoire.

@hebernardEquisoft hebernardEquisoft changed the title chore(tabs): add default selected tab option chore(tabs)!: add default selected tab option May 30, 2024
@meriouma meriouma changed the title chore(tabs)!: add default selected tab option feat(tabs)!: add default selected tab option Jun 6, 2024
@meriouma meriouma changed the title feat(tabs)!: add default selected tab option feat(Tabs)!: add default selected tab option Jun 6, 2024
@hebernardEquisoft hebernardEquisoft merged commit 8c66ac4 into master Jun 7, 2024
23 checks passed
@hebernardEquisoft hebernardEquisoft deleted the dev/DS-1155 branch June 7, 2024 12:49
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.

4 participants