Skip to content

Conversation

@chrisolsen
Copy link
Collaborator

@chrisolsen chrisolsen commented Oct 17, 2025

When changing dynamically bound dropdown items, if the items are reduced and no new items are added, the dropdown items would not change.

Before (the change)

  • Bind the values [red, green, blue], view the dropdown items to see the 3 items
  • Bind the value [red], you will still see the three previously bound items

After (the change)

  • Bind the values [red, green, blue], view the dropdown items to see the 3 items

  • Bind the value [red], you will now just see red

  • Browser tests written

  • Tested in React

  • Tested in Angular

  • Tested Web components

Steps needed to test

Test code exists in the prs folder => 2333

@chrisolsen chrisolsen changed the base branch from alpha to dev November 5, 2025 22:08
@ArakTaiRoth ArakTaiRoth linked an issue Nov 19, 2025 that may be closed by this pull request
@chrisolsen chrisolsen force-pushed the chris/2333-dropdown-reset branch from cb1d528 to 21662a4 Compare November 27, 2025 19:56
@chrisolsen chrisolsen requested review from ArakTaiRoth, twjeffery and willcodeforcoffee and removed request for twjeffery November 27, 2025 19:57
@chrisolsen chrisolsen marked this pull request as ready for review November 27, 2025 19:59
@chrisolsen chrisolsen force-pushed the chris/2333-dropdown-reset branch 2 times, most recently from f8387bd to 8b474ca Compare November 27, 2025 20:08
Copy link
Collaborator

@willcodeforcoffee willcodeforcoffee left a comment

Choose a reason for hiding this comment

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

LGTM

@ArakTaiRoth ArakTaiRoth linked an issue Dec 3, 2025 that may be closed by this pull request
@chrisolsen chrisolsen force-pushed the chris/2333-dropdown-reset branch from 23d242f to 62d4a9e Compare December 4, 2025 15:44
ArakTaiRoth
ArakTaiRoth previously approved these changes Dec 4, 2025
@chrisolsen chrisolsen changed the title fix(#2333): ensure dropdown items are kept in sync when changed fix(#3248): ensure dropdown items are kept in sync when changed Dec 4, 2025
@chrisolsen chrisolsen changed the title fix(#3248): ensure dropdown items are kept in sync when changed fix(#3248): sync the dropdown values when children are added/removed Dec 4, 2025
Copy link
Collaborator

@willcodeforcoffee willcodeforcoffee left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@chrisolsen chrisolsen merged commit 18a21fb into dev Dec 4, 2025
4 of 5 checks passed
@chrisolsen chrisolsen deleted the chris/2333-dropdown-reset branch December 4, 2025 19:53
@tzuge
Copy link
Collaborator

tzuge commented Dec 4, 2025

🎉 This PR is included in version 1.40.0-dev.12 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tzuge
Copy link
Collaborator

tzuge commented Dec 4, 2025

🎉 This PR is included in version 6.10.0-dev.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dropdown not properly syncing

5 participants