Skip to content
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

feat(kmultiselect): add search-placeholder prop [KHCP-13352] #2419

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

portikM
Copy link
Member

@portikM portikM commented Sep 27, 2024

Summary

Jira: https://konghq.atlassian.net/browse/KHCP-13352

Changes (KMultiselect):

  • add searchPlaceholder prop that acts as search input placeholder
    • in collapsed-context mode - the trigger element is the search input so text passed through searchPlaceholder prop displays in the trigger element when no items are selected
    • otherwise, searchPlaceholder sets a placeholder in search input inside multiselect dropdown
    • in non-collapsed-context mode, existing placeholder prop (if has value) is used as trigger element text when no items are selected
  • add placeholder and searchPlaceholder props documentation

@portikM portikM self-assigned this Sep 27, 2024
@portikM portikM requested a review from a team as a code owner September 27, 2024 17:54
Copy link

netlify bot commented Sep 27, 2024

Deploy Preview for kongponents-sandbox ready!

Name Link
🔨 Latest commit dd386b1
🔍 Latest deploy log https://app.netlify.com/sites/kongponents-sandbox/deploys/66f6f15f603dbe000875d5f0
😎 Deploy Preview https://deploy-preview-2419--kongponents-sandbox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Sep 27, 2024

Deploy Preview for kongponents ready!

Name Link
🔨 Latest commit dd386b1
🔍 Latest deploy log https://app.netlify.com/sites/kongponents/deploys/66f6f15f903ce20008a8b88d
😎 Deploy Preview https://deploy-preview-2419--kongponents.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

searchPlaceholder: {
type: String,
default: '',
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Shall we warn users when collapsedContext is true but placeholder is not empty?

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't think that's necessary: the default displayed text for both modes is 0 items selected - I don't think we need to specifically point that out.

@portikM portikM enabled auto-merge (squash) September 30, 2024 14:57
Copy link
Contributor

@Justineo Justineo left a comment

Choose a reason for hiding this comment

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

LGTM

@portikM portikM merged commit 4062052 into main Sep 30, 2024
11 checks passed
@portikM portikM deleted the feat/khcp-13352-kmultiselect-placeholders branch September 30, 2024 17:12
kongponents-bot pushed a commit that referenced this pull request Sep 30, 2024
# [9.9.0](v9.8.4...v9.9.0) (2024-09-30)

### Features

* **kmultiselect:** add search-placeholder prop [KHCP-13352] ([#2419](#2419)) ([4062052](4062052))
@kongponents-bot
Copy link
Collaborator

🎉 This PR is included in version 9.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@kongponents-bot
Copy link
Collaborator

Preview package from this PR in consuming application

In consuming application project install preview version of kongponents generated by this PR:

@kong/kongponents@pr-2419

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.

4 participants