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.

Sign up for GitHub

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

Jump to bottom

[dropdown] onChange does not fire when pasting a value in multiple search dropdown. #2764

Closed
tei6 opened this issue Apr 22, 2023 · 2 comments
Closed

[dropdown] onChange does not fire when pasting a value in multiple search dropdown. #2764

tei6 opened this issue Apr 22, 2023 · 2 comments
Labels
type/bug Any issue which is a bug or PR which fixes a bug
Milestone
2.9.3

Comments

@tei6
Copy link

tei6 commented Apr 22, 2023

Bug Report

onChange does not fire when pasting a value from the clipboard in multiple search dropdown.

Steps to reproduce

  1. Create multiple search dropdown.
  2. Paste choice values (e.g., "AL" or "AL, AK") into a multiple search dropdown.

Expected result

It is expected that onChange will fire when a value is pasted,
just as onChange will fire when a selection is made by clicking on a choice.

Playground

https://jsfiddle.net/m7x18fvr/1/

Version

2.9.2
@tei6 tei6 added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels Apr 22, 2023
@lubber-de lubber-de mentioned this issue Apr 23, 2023
Merged
@lubber-de
Copy link
Member

Fixed by #2765
See your adjusted jsfiddle here https://jsfiddle.net/lubber/13e0pn8q/

@lubber-de lubber-de added state/has-pr An issue which has a related PR open and removed state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged labels Apr 23, 2023
@lubber-de lubber-de added this to the 2.9.3 milestone Apr 23, 2023
@tei6
Copy link
Author

tei6 commented Apr 23, 2023

@lubber-de
I can confirm that the issue has been fixed. Thank you for your prompt response!

@lubber-de lubber-de added tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build and removed state/has-pr An issue which has a related PR open labels Apr 28, 2023
@lubber-de lubber-de removed the tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build label Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Milestone
2.9.3
Development

No branches or pull requests
2 participants
@tei6 @lubber-de