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
[DOCS] Hide duplicate tabs #9570
Conversation
✅ Deploy Preview for niobium-lead-7998 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -8,6 +8,10 @@ div[role="tabpanel"]{ | |||
font-weight: var(--ifm-font-weight-normal); | |||
line-height: var(--p-line-height); | |||
|
|||
&.hidden { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we going to need to add this class to each screen in order to get this effect?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My understanding is that we'll add the class name to any Tab group that we want to hide the tab selection portion of.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rachel is right, any tab that has the hidden class in the markdown will be hidden!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽
Description
If
className="hidden"
is added to a Tabs component, it will have its tabs hidden (useful for nested tabs inside the same document that will always have the same value).You can check this does not break the logic of switching tabs, there's an example in this page: https://deploy-preview-9570.docs.greatexpectations.io/docs/oss/guides/setup/installation/install_gx. If you select
pip
orconda
tabs inside thelocal
tab, you will see the content is updated properly, even though the second set of tabs that previously showed optionspip
andconda
again is hidden.Screenshots
Before
After
invoke lint
(usesblack
+ruff
)For more information about contributing, see Contribute.
After you submit your PR, keep the page open and monitor the statuses of the various checks made by our continuous integration process at the bottom of the page. Please fix any issues that come up and reach out on Slack if you need help. Thanks for contributing!