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
fix: detach should be sent for node (#17695) (CP: 23.5) #19051
Conversation
Do not clean the node id or hasAttached state too early as then we do not send the detach event for the node. Fixes #17653
This PR is eligible for auto-merging policy, so it has been approved automatically. If there are pending conditions, auto merge (with 'squash' method) has been enabled for this PR[Message is sent from bot] |
) Enables moving a component to another UI by just calling Element.removeFromTree() and then adding the component to the other UI. Fixes #18390 --------- Co-authored-by: caalador <mikael.grankvist@vaadin.com> Co-authored-by: Tatu Lund <tatu@vaadin.com>
Quality Gate passedIssues Measures |
This PR is eligible for auto-merging policy, so it has been approved automatically. If there are pending conditions, auto merge (with 'squash' method) has been enabled for this PR[Message is sent from bot] |
This ticket/PR has been released with Vaadin 23.5.0.alpha1 and is also targeting the upcoming stable 23.5.0 version. |
No description provided.