Skip to content

feat(Tabs): add the option to disable the Add button #11872

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

Merged
merged 2 commits into from
Jun 17, 2025

Conversation

ksiekl
Copy link
Contributor

@ksiekl ksiekl commented Jun 17, 2025

Closes #11873

This PR adds the option to make the + button in the Tabs component disabled, if the isAddButtonDisabled prop is set.

Although the change in the code is really small, I'm not fully familiar with the project and couldn't make it run locally to check visually how the component behaves 😢 . Feel free to confirm that it behaves as expected. All tests pass on my end, so it should be good to go.

@patternfly-build
Copy link
Contributor

patternfly-build commented Jun 17, 2025

The button in the Tabs component for adding new tabs can now get disabled if the isAddButtonDisabeled prop is set.
@ksiekl ksiekl force-pushed the isAddButtonDisabled branch from 0aa591a to 147d246 Compare June 17, 2025 14:37
@ksiekl
Copy link
Contributor Author

ksiekl commented Jun 17, 2025

Thanks @thatblindgeye for the comments!

@ksiekl ksiekl requested a review from thatblindgeye June 17, 2025 14:39
@thatblindgeye thatblindgeye merged commit 1337e36 into patternfly:main Jun 17, 2025
13 checks passed
@patternfly-build
Copy link
Contributor

Your changes have been released in:

Thanks for your contribution! 🎉

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.

Tabs - allow 'add tabs' button to be disabled conditionally
4 participants