-
Notifications
You must be signed in to change notification settings - Fork 6.8k
feat(drag-drop): add API to get/set current position of a standalone draggable #14696
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
feat(drag-drop): add API to get/set current position of a standalone draggable #14696
Conversation
3fb0b3c
to
8f88ce2
Compare
32610e9
to
6402dbe
Compare
The feedback has been addressed @jelbourn. |
Hi @crisbeto! This PR has merge conflicts due to recent upstream merges. |
d254ea5
to
8de3187
Compare
8de3187
to
115b500
Compare
Really waiting to get this feature! |
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
82a0cf8
to
13025ad
Compare
13025ad
to
c03077f
Compare
c03077f
to
4faa61b
Compare
any update? |
4faa61b
to
2016c25
Compare
Hi, I have been watching for this change to be released since January, I know this is targeting a minor release rather than a patch release but I cannot find any indication or timetable for when the next minor release might be. Is there a public facing plan or schedule somewhere or some indication of what needs to be completed before the next minor release (or release that this will be included in)? |
2016c25
to
82a2cbf
Compare
I'm really grateful for this feature, I've run into this problem and looking forward to seeing this solution. Is there any update as to when this will be available ? |
A branch-manger issue is blocking merging @jelbourn @devversion. Can this be fixed so that this PR can be merged |
…draggable Adds an API that allows the consumer to get the current position of a standalone draggable and to set it. This is useful for cases where the dragged position should be preserved when the user navigates away and then restored when they return. Fixes angular#14420. Fixes angular#14674.
82a2cbf
to
959c79b
Compare
When guys you plan to release get/set position fixes, we are waiting for it. |
@MDzyga can you post an example of what you're doing? Also it would be better to open an issue. |
@MDzyga ist right, setting the position doesn't work. I've tried setting the freeDragPosition object and called change detection afterwards, but nothing happens :( |
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. |
Adds an API that allows the consumer to get the current position of a standalone draggable and to set it. This is useful for cases where the dragged position should be preserved when the user navigates away and then restored when they return.
Fixes #14420.
Fixes #14674.