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

fix(tab):generic id type added to TabItem #223

Merged
merged 3 commits into from
Mar 26, 2024

Conversation

tatata96
Copy link
Contributor

@tatata96 tatata96 changed the base branch from main to next-release January 18, 2023 07:42
@github-actions
Copy link

github-actions bot commented Jan 18, 2023

PR Preview Action v1.2.0
🚀 Deployed preview to https://Hipo.github.io/react-ui-toolkit/pr-preview/pr-223/
on branch gh-pages at 2023-02-02 08:40 UTC

Copy link
Contributor

@gulcinuras gulcinuras left a comment

Choose a reason for hiding this comment

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

@tatata96 I asked a small change, but we can also discuss if that makes sense or not 😸
Also, I noticed that in 12-Tab.stories we are not providing the initialState in a Record type, so that it gives a type error.

Therefore, I think we can update the initialState on line 39 like this:

<StateProvider initialState={{index: 0}}>

and update the other places we use that state.

src/tab/Tab.tsx Outdated Show resolved Hide resolved
src/tab/Tab.tsx Outdated Show resolved Hide resolved
@tatata96 tatata96 added Ready for new release PR is reviewed and ready to be merged and removed ready for review labels Apr 18, 2023
Copy link
Contributor

@yasincaliskan yasincaliskan left a comment

Choose a reason for hiding this comment

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

🎉

src/tab/Tab.tsx Show resolved Hide resolved
stories/12-Tab.stories.tsx Show resolved Hide resolved
src/tab/header/item/TabHeaderItem.tsx Show resolved Hide resolved
src/tab/Tab.tsx Show resolved Hide resolved
@jamcry jamcry merged commit 1cbf0cf into next-release Mar 26, 2024
@jamcry jamcry deleted the fix/tab-generic-type-id branch March 26, 2024 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for new release PR is reviewed and ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants