Skip to content

fix(pat-contentbrowser): don't fire change while rendering the initial selection - #1641

Open
MrTango wants to merge 1 commit into
masterfrom
pr/contentbrowser-initial-selection
Open

MrTango wants to merge 1 commit into
masterfrom
pr/contentbrowser-initial-selection

Conversation

@MrTango

@MrTango MrTango commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

The selected-items effect wrote the initial value back to the input on mount and dispatched a change event for it, so pat-formunloadalert marked every untouched form with a content browser field as edited and warned when leaving the page.

Only write and dispatch when the value actually differs.

@MrTango
MrTango requested a review from petschki September 16, 2026 09:25
…l selection

The selected-items effect wrote the initial value back to the input on mount
and dispatched a change event for it, so pat-formunloadalert marked every
untouched form with a content browser field as edited and warned when
leaving the page.

Only write and dispatch when the value actually differs.
@petschki
petschki force-pushed the pr/contentbrowser-initial-selection branch from 4829682 to 2ae9d77 Compare September 18, 2026 08:27
@petschki

Copy link
Copy Markdown
Member

/run-coredev-6.3-robottests

This branch has not been deployed

No deployments
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.

2 participants