Skip to content

fix(ui5-multi-combobox): fix token deletion when click event is lost due to re-render#13530

Merged
Vonahz merged 4 commits into
mainfrom
multicombobox_bug
May 21, 2026
Merged

fix(ui5-multi-combobox): fix token deletion when click event is lost due to re-render#13530
Vonahz merged 4 commits into
mainfrom
multicombobox_bug

Conversation

@Vonahz

@Vonahz Vonahz commented May 15, 2026

Copy link
Copy Markdown
Contributor

Fixes: #13514

Token deletion would fail intermittently when clicking the delete icon
because the mousedown event set toBeDeleted=true, which could trigger
a re-render that replaced the delete icon DOM element before the click
event fired, causing the click event to be lost and preventing deletion.

This was most easily reproduced when tokens were collapsed/overflown,
but could happen in various scenarios where DOM updates occurred between
mousedown and click events.

@ui5-webcomponents-bot

ui5-webcomponents-bot commented May 15, 2026

Copy link
Copy Markdown
Collaborator

🧹 Preview deployment cleaned up: https://pr-13530--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 15, 2026 15:52 Inactive
@aleksandar-terziev aleksandar-terziev self-requested a review May 18, 2026 18:53
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 12:39 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 13:01 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 17:31 Inactive
@Vonahz Vonahz merged commit c1abfb0 into main May 21, 2026
13 checks passed
@Vonahz Vonahz deleted the multicombobox_bug branch May 21, 2026 13:58
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 21, 2026 13:58 Inactive
@ui5-webcomponents-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version v2.23.0-rc.0 🎉

The release is available on v2.23.0-rc.0

Your semantic-release bot 📦🚀

@sap-ui5-webcomponents-release

Copy link
Copy Markdown

🎉 This PR is included in version v2.23.0 🎉

The release is available on v2.23.0

Your semantic-release bot 📦🚀

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.

[MultiComboBox|Tokenizer]: Component fails to remove item from selection list and collapses instead

3 participants