Skip to content

Conversation

@nicolethoen
Copy link
Contributor

No description provided.

@patternfly-build
Copy link
Collaborator

patternfly-build commented Jun 12, 2023

@nicolethoen nicolethoen requested a review from mcoker June 13, 2023 00:53
#ws-html-deprecated-c-select-loading-menu-item,
#ws-html-deprecated-c-select-placeholder-expanded,
#ws-html-deprecated-c-select-placeholder-item-disabled,
#ws-html-deprecated-c-select-placeholder-item-enabled {
Copy link
Contributor

Choose a reason for hiding this comment

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

@ashley-o0o noticed an example is missing a height today - https://patternfly-pr-5667.surge.sh/components/menus/select#multi-with-typeahead-invalid

A wee bit of scope creep, but could we sneak that fix in here? The ID for that example is ws-html-deprecated-c-select-multi-with-typeahead-invalid

Suggested change
#ws-html-deprecated-c-select-placeholder-item-enabled {
#ws-html-deprecated-c-select-placeholder-item-enabled,
#ws-html-deprecated-c-select-multi-with-typeahead-invalid {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure thing

@mcoker mcoker merged commit 56f8ff5 into patternfly:main Jun 13, 2023
@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 5.0.0-prerelease.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.0.0-alpha.1 🎉

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.

3 participants