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

Tessa/minimal tab suggestions #1414

Merged
merged 6 commits into from
Jun 23, 2023

Conversation

tesk9
Copy link
Contributor

@tesk9 tesk9 commented Jun 22, 2023

Suggestions for #1412 and the minimalist tab example on the component catalog.

Mostly, I want the example to be more minimal! So I've cut out the Panel pangrams and made the tabHtml example more closely matched to the intended use of tabHtml.

I've also fixed some of the spacing in the example code by using Code helpers (I know the Tabs component that we started out copying doesn't provide great examples -- sorry about that!

After

Screen Shot 2023-06-22 at 5 04 16 PM Screen Shot 2023-06-22 at 5 04 12 PM

@tesk9 tesk9 requested a review from ap-nri June 22, 2023 23:04
@tesk9 tesk9 mentioned this pull request Jun 22, 2023
12 tasks
@tesk9 tesk9 requested a review from bendansby June 22, 2023 23:15
@bendansby
Copy link
Contributor

I wonder if we should rename the component to TabsMinimal so they're grouped in various alphabetical situations?

@tesk9
Copy link
Contributor Author

tesk9 commented Jun 22, 2023

Oh dang! I love that idea! 😍 (Will leave to @ap-nri to actually implement though, if he's on board with the change)

@bendansby
Copy link
Contributor

I'd also like to do this #1330 for TabsMinimal as well (except 15px based on Ravi's design) so we have that standardized off the bat.

@ap-nri
Copy link
Contributor

ap-nri commented Jun 23, 2023

yes and yes and yes I love all of these you are strong and wise collaborators tysm @bendansby and @tesk9

@ap-nri ap-nri merged commit 0db8878 into growth/minimalist-tabs Jun 23, 2023
@ap-nri ap-nri deleted the tessa/minimal-tab-suggestions branch June 23, 2023 15:20
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.

3 participants