Skip to content

Conversation

@dlabaj
Copy link
Collaborator

@dlabaj dlabaj commented Jan 12, 2024

Closes #87
RHCLOUD-30381

Added the close button from OpenShift.

Screenshot 2024-01-12 at 9 31 40 AM

@dlabaj dlabaj added the enhancement New feature or request label Jan 12, 2024
@dlabaj dlabaj requested a review from fhlavac January 12, 2024 14:39
@dlabaj dlabaj self-assigned this Jan 12, 2024
@fhlavac
Copy link
Contributor

fhlavac commented Jan 15, 2024

To further customize a close button, you can also utilize all properties of the button component, with the exception of variant.

In the current state you still may be able to overwrite variant using props. But, not sure if it's a problem. We can let user of our component do that - so maybe just remove the note from the docs?

@fhlavac
Copy link
Contributor

fhlavac commented Jan 15, 2024

Just some nitpicking.
@dlabaj looks good!

Copy link
Contributor

@nicolethoen nicolethoen left a comment

Choose a reason for hiding this comment

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

only one note from me

@fhlavac
Copy link
Contributor

fhlavac commented Jan 31, 2024

@dlabaj would you have time to take a look today so we can get this one in? Thank you 🙂

@dlabaj dlabaj merged commit 0b38a2a into patternfly:main Jan 31, 2024
@github-actions
Copy link

🎉 This PR is included in version 5.1.0-prerelease.4 🎉

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Add Close Button

3 participants