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

Support icons in selected item also, not only in list of options #337

Merged
merged 9 commits into from
Jun 13, 2024

Conversation

AlexeySemenko
Copy link
Contributor

added config option selectedLabelRenderer, same as labelRenderer to allow showing icons or other custom content in selected item when the select is closed

…llow showing icons or other custom content in selected item when the select is closed
@gnbm gnbm requested a review from sa-si-dev June 4, 2024 21:41
@gnbm
Copy link
Collaborator

gnbm commented Jun 4, 2024

@AlexeySemenko can you please add more context to the PR, including running tests and maybe including tests runs (example, something more aligned with this PR)

dist/virtual-select.js Outdated Show resolved Hide resolved
package-lock.json Outdated Show resolved Hide resolved
@AlexeySemenko
Copy link
Contributor Author

AlexeySemenko commented Jun 5, 2024

After adding the selectedLabelRenderer property
image

  • possible to add icons to selected items as well as in items list

  • Added cypress relevant test - has flag icon on selected item
    image

  • Added documentation

image

docs/examples.md Outdated Show resolved Hide resolved
Co-authored-by: Arnaud Benhamdine <[email protected]>
dist-archive/virtual-select-1.0.44.min.js Outdated Show resolved Hide resolved
dist/virtual-select.js Outdated Show resolved Hide resolved
dist/virtual-select.min.js Outdated Show resolved Hide resolved
@AlexeySemenko AlexeySemenko requested a review from gnbm June 5, 2024 17:23
Copy link
Collaborator

@gnbm gnbm left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

@gnbm gnbm requested a review from abenhamdine June 5, 2024 17:50
@gnbm
Copy link
Collaborator

gnbm commented Jun 10, 2024

@AlexeySemenko once you resolve the conflicts the PR should be good to go and get merged 👍🏻

@AlexeySemenko
Copy link
Contributor Author

@AlexeySemenko once you resolve the conflicts the PR should be good to go and get merged 👍🏻

conflict resolved

@gnbm gnbm merged commit 5a6ab6a into sa-si-dev:master Jun 13, 2024
@gnbm gnbm mentioned this pull request Aug 26, 2024
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.

3 participants