Removing a Draggable during a drag should work #6341
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
Hixie
reviewed
Oct 15, 2016
Hixie
reviewed
Oct 15, 2016
Contributor
There was a problem hiding this comment.
please add a comment here or in _disposeRecognizerIfInactive or at the class level discussing the unusual lifetime conventions of this object and the _recognizer it creates.
Contributor
|
LGTM. There's a comment somewhere in drag_target.dart that talks about lifetime that you may need to adjust as well. |
Previously we would maroon the feedback in the overlay. Now we let the drag proceed and only tear down the gesture recognizer when all the drags it spawns have been resolved. Fixes flutter#6151
abarth
force-pushed
the
remove_draggable_during_drag
branch
from
7426fbd to
608cec4
Compare
a-siva
added a commit
that referenced
this pull request
Sep 27, 2018
Contains the following changes: d80c1de (HEAD -> master, upstream/master) Roll src/third_party/skia ab3144c3abb9..656cefe65d62 (11 commits) (#6362) edf6249 Add pushOffset to SceneBuilder (#6349) 3a01f39 Change log level from ERROR to WARNING (#6361) 5ae4708 Roll src/third_party/skia 227d4e10276c..ab3144c3abb9 (11 commits) (#6360) 763627f Do not export libdart symbols (#6337) 3052dbd SystemNavigator.pop can pop w/o UINavigationController (#6341) 0c096f7 Roll src/third_party/skia b3e48afc936d..227d4e10276c (1 commits) (#6359) b8c2a17 Roll src/third_party/skia cfe1264d7465..b3e48afc936d (3 commits) (#6356) c589b31 Expose push/popRoute on FlutterViewController (#6347) 075b3fc Roll src/third_party/skia 5ea41fc89b26..cfe1264d7465 (1 commits) (#6355) 2dd9b99 Roll Dart to version 808ed6238b9262660e31ea826f7aea6cfa3a3493 (#6354) 5b79938 Dont make any binaries specify an X11 dependency. (#6353) 309ac4e V0.8.2 fix compile problem with xcode10 (#6339) 26fdd1e Roll src/third_party/skia 5767fc042834..5ea41fc89b26 (3 commits) (#6351) cc44ca5 Perform persistent cache stores on the IO thread outside the frame workload. (#6350) f2a3df9 Wire up the Skia persistent GPU related artifacts cache. (#6278)
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously we would maroon the feedback in the overlay. Now we let the
drag proceed and only tear down the gesture recognizer when all the
drags it spawns have been resolved.
Fixes #6151