-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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(drag-drop): not transferring input value when cloning element for preview #20009
fix(drag-drop): not transferring input value when cloning element for preview #20009
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This transferData
routine is only applied to descendant nodes, but it also should check the top-level node itself (e.g. if a cdkDrag
directive is directly placed on a select
element).
… preview We use `cloneNode` to create a preview for an element which clones all of the attributes of an element, but that may not pick up properties like its `value`. These changes add some logic where we clone the value with a similar approach that we use for cloning `canvas` elements. Also generalizes the approach a bit so we don't have to keep repeating the logic. Fixes angular#19905.
238293b
to
47cd67e
Compare
Good point. Updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
… preview (#20009) We use `cloneNode` to create a preview for an element which clones all of the attributes of an element, but that may not pick up properties like its `value`. These changes add some logic where we clone the value with a similar approach that we use for cloning `canvas` elements. Also generalizes the approach a bit so we don't have to keep repeating the logic. Fixes #19905. (cherry picked from commit d53d5b6)
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
We use
cloneNode
to create a preview for an element which clones all of the attributes of an element, but that may not pick up properties like itsvalue
. These changes add some logic where we clone the value with a similar approach that we use for cloningcanvas
elements. Also generalizes the approach a bit so we don't have to keep repeating the logic.Fixes #19905.