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
fix(button): fixes primary background in pf-t-dark block #5652
Conversation
Preview: https://patternfly-pr-5652.surge.sh A11y report: https://patternfly-pr-5652-a11y.surge.sh |
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.
LGTM!
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.
LPTM 👍
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.
👍 lgtm
🎉 This PR is included in version 5.0.0-prerelease.4 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 6.0.0-alpha.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
fixes #5436
This PR addresses the custom primary and secondary button styles that were removed from the
pf-t-dark
placeholder in #5435You can see what the old styles looked like when a primary or secondary button was used in a "dark" block, which were based off of an old design here - https://www.patternfly.org/v4/components/masthead/html#basic-with-mixed-content
This PR just re-introduces a var override in the placeholder that sets the primary button background, and resets it in dark theme's placeholder. The secondary button is already accessible in a dark block, so no override is necessary. Here's that demo in this PR - https://patternfly-pr-5652.surge.sh/components/masthead#basic-with-mixed-content