Skip to content

[DOCS] - Clarify FluentSelect SelectedOptionsChanged not triggered with manual options (#3670 and #1207) #3680

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

Merged
merged 1 commit into from
Apr 15, 2025

Conversation

pedro-cons
Copy link
Contributor

Clarify FluentSelect SelectedOptionsChanged not triggered with manual options (#3670 and #1207)

📖 Description

This PR improves the documentation of the FluentSelect component by adding a clarification about the SelectedOptionsChanged event.

When using manual options, FluentSelect's internal data structure cannot be reliably updated. Because of that, the SelectedOptionsChanged callback will not be triggered.
This is a known limitation, and we do not foresee changes to this behavior in the short term.

🎫 Issues

Fixes #3670 and #1207

📑 Test Plan

  • Validated that the updated note appears correctly in the FluentSelect documentation page.
  • Confirmed that no visual or structural issues were introduced.

Updated documentation:

Preview

Let me know if you’d like me to include any additional details.

Thanks!

@vnbaaij
Copy link
Collaborator

vnbaaij commented Apr 15, 2025

Hi, Thanks for your contribution. Once you accept the CLA, this can be merged in.

@pedro-cons
Copy link
Contributor Author

@microsoft-github-policy-service agree

@vnbaaij vnbaaij enabled auto-merge (squash) April 15, 2025 06:12
@pedro-cons
Copy link
Contributor Author

Hi, Thanks for your contribution. Once you accept the CLA, this can be merged in.

Hi! I've just accepted the CLA

Thanks!

@vnbaaij vnbaaij merged commit 4612a28 into microsoft:dev Apr 15, 2025
4 checks passed
@pedro-cons pedro-cons deleted the docs/event-manual-options branch April 15, 2025 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docs: fix/add FluentSelect SelectedOptionsChanged event not trigger when using manual options
2 participants