Skip to content

fix(ui5-toolbar-select): add invalidateOnChildChange#12676

Merged
NakataCode merged 3 commits into
mainfrom
toolbar-select-option
Dec 2, 2025
Merged

fix(ui5-toolbar-select): add invalidateOnChildChange#12676
NakataCode merged 3 commits into
mainfrom
toolbar-select-option

Conversation

@NakataCode

@NakataCode NakataCode commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Problem:

When programmatically changing the selected property on a ToolbarSelectOption, the ToolbarSelect component's display was not updating to reflect the change.

This occurred because the ToolbarSelect was not invalidating and re-rendering when its child options' properties changed.


Solution:

  • Added invalidateOnChildChange: true to the options slot decorator in ToolbarSelect to trigger re-rendering when child ToolbarSelectOption properties change
  • Also added cypress test

Fixes: #12619

@ui5-webcomponents-bot

ui5-webcomponents-bot commented Nov 14, 2025

Copy link
Copy Markdown
Collaborator

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

@ui5-webcomponents-bot
ui5-webcomponents-bot temporarily deployed to preview November 14, 2025 11:14 Inactive
@NakataCode
NakataCode marked this pull request as ready for review November 14, 2025 11:16
@NakataCode NakataCode changed the title fix(ui5-toolbar-select): sync value when option selected property changes programmatically fix(ui5-toolbar-select): add invalidateOnChildChange Nov 27, 2025
@ui5-webcomponents-bot
ui5-webcomponents-bot temporarily deployed to preview November 27, 2025 12:20 Inactive
@NakataCode
NakataCode merged commit 23a2df8 into main Dec 2, 2025
14 checks passed
@NakataCode
NakataCode deleted the toolbar-select-option branch December 2, 2025 12:22
@ui5-webcomponents-bot
ui5-webcomponents-bot temporarily deployed to preview December 2, 2025 12:23 Inactive
@ui5-webcomponents-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version v2.17.0-rc.5 🎉

The release is available on v2.17.0-rc.5

Your semantic-release bot 📦🚀

@ui5-webcomponents-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version v2.17.0 🎉

The release is available on v2.17.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.

[ToolbarSelect]: ToolbarSelect display value does not display selected option

3 participants