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):Separate tab body #225

Closed
wants to merge 2 commits into from
Closed

Conversation

tatata96
Copy link
Contributor

@tatata96 tatata96 self-assigned this Jan 20, 2023
@tatata96 tatata96 changed the base branch from main to next-release January 20, 2023 09:36
@github-actions
Copy link

PR Preview Action v1.2.0
🚀 Deployed preview to https://Hipo.github.io/react-ui-toolkit/pr-preview/pr-225/
on branch gh-pages at 2023-01-20 09:38 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 as far as I understand from the mentioned issue, we should have a compound components (could be named as TabHeader and TabBody) which would allow us to customize easier. So, I think we should follow a similar atomized approach as we have in the Select component. What do you think?

Copy link
Contributor

@ErenYLCN ErenYLCN left a comment

Choose a reason for hiding this comment

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

I agree with @gulcinuras here, I think we should at least have a TabHeader component to completely fulfill the issue

@tatata96 tatata96 added W.I.P Work in progress and removed ready for review labels Jan 27, 2023
@yasincaliskan
Copy link
Contributor

I also agree with @gulcinuras. We should have TabHeader and TabBody compound components. We can also check Toggle component, it is simpler than Select.

@jamcry jamcry closed this Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
W.I.P Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants