E530 chore(tab-content): move tab content under tabs by mcoker · Pull Request #7381 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

chore(tab-content): move tab content under tabs#7381

Merged
mcoker merged 3 commits intopatternfly:mainfrom
mcoker:issue-7328
Mar 11, 2025
Merged

chore(tab-content): move tab content under tabs#7381
mcoker merged 3 commits intopatternfly:mainfrom
mcoker:issue-7328

Conversation

@mcoker
Copy link
Contributor
@mcoker mcoker commented Mar 5, 2025

fixes #7328

I left the tabs example titles alone in case someone saved/bookmarked them. Lemme know if you'd prefer update any of them (for example "Default" to "Default tabs", or under tab content "Secondary" to "Secondary tab content").

@patternfly-build
Copy link
Collaborator
patternfly-build commented Mar 5, 2025

Copy link
Contributor
@thatblindgeye thatblindgeye left a comment

Choose a reason for hiding this comment

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

Personally it's probably fine for now re: titling. The titles are unique on the page so it should e fine.

Eventually it might be good to rethink the naming convention for examples and the structures, but waiting until we get the new docs framework up would be better since we may have more flexibility with how we can structure things (e.g. maybe something like ## Tab examples > ### Basic > the example itself if there's really only 1 example (rather than "Examples" > "Default" > "Default tab example" > the example itself), and ## Tab examples > ### Overflow > #### Beginning of list > the example itself etc).

But again, not something to do here either way, just thinking out loud.

Copy link
Collaborator
@andrew-ronaldson andrew-ronaldson left a comment

Choose a reason for hiding this comment

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

Good to go with @thatblindgeye points

@mcoker mcoker merged commit 8686eea into patternfly:main Mar 11, 2025
4 checks passed
@mcoker mcoker deleted the issue-7328 branch March 11, 2025 01:58
@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.2.0-prerelease.18 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TabContent - update documentation to be part of Tabs docs

4 participants

0