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

[dropdown] Multiple item drop down with HTML special chars not properly working anymore #2979

Open
mkrecek234 opened this issue Jan 18, 2024 · 1 comment
Assignees
Labels
tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build type/bug Any issue which is a bug or PR which fixes a bug
Milestone

Comments

@mkrecek234
Copy link

mkrecek234 commented Jan 18, 2024

Bug Report

I have a multiple dropdown which has items looking like Firstname Lastname <email@email.de> both as values and titles.
With the most recent versions, this broke, and only the first item is being displayed, even though the input has multiple items in it.

Steps to reproduce


<div id="_f7a9797c367f6416e0803bf7b442bddb__out_form_layout_recipient" class="ui input eight wide">

<div class="ui dropdown multiple search selection " style="width: 100%;">
<input form="_072d8b31e9c87b977c5bdeca6446a7d9__tualpage_messageform_form" name="recipient" type="hidden" id="_f7a9797c367f6416e0803bf7b442bddb__out_form_layout_recipient_input" value="Larry Finton &lt;sales@best-distributor.com&gt;,Frank Smith &lt;ceo@best-distributor.com&gt;">
<i class="dropdown icon"></i>
<input form="_072d8b31e9c87b977c5bdeca6446a7d9__tualpage_messageform_form" class="search"  style="width: 100%;">
<div class="default text">Empfängeradressen</div>
<div class="menu">
<div class="item" data-value="Larry Finton &lt;sales@best-distributor.com&gt;">Larry Finton &lt;sales@best-distributor.com&gt;</div>
<div class="item" data-value="Frank Smith &lt;ceo@best-distributor.com&gt;">Frank Smith &lt;ceo@best-distributor.com&gt;</div>

</div>
</div>

</div>

Expected result

The dropdown should show BOTH entries, Larry and Frank.

Actual result

It only shows the first.

Testcase

https://jsfiddle.net/nae8ypo3/1/

Screenshot (if possible)

image

Version

Latest

@mkrecek234 mkrecek234 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 Jan 18, 2024
@lubber-de lubber-de self-assigned this Jan 26, 2024
@lubber-de lubber-de removed the state/awaiting-triage Any issues or pull requests which haven't yet been triaged label Jan 26, 2024
@lubber-de
Copy link
Member

Fixed by #2993
See your adjusted jsfiddle here https://jsfiddle.net/lubber/40qm6krt/

@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/awaiting-investigation Anything which needs more investigation labels Feb 9, 2024
@lubber-de lubber-de added this to the 2.9.4 milestone Feb 9, 2024
@lubber-de lubber-de added state/has-pr An issue which has a related PR open tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build and removed tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build state/has-pr An issue which has a related PR open labels Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Development

No branches or pull requests

2 participants