-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Add hitTestBehavior and rootOverlay to LongPressDraggable constructor… #146386
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
Add hitTestBehavior and rootOverlay to LongPressDraggable constructor… #146386
Conversation
351adbb to
429f4c8
Compare
goderbauer
left a comment
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
justinmc
left a comment
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 with a nit, thanks for the PR!
429f4c8 to
9ebf730
Compare
justinmc
left a comment
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.
Indentation looks good now, thanks. Renewing my LGTM 👍
Manual roll requested by [email protected] flutter/flutter@2e748e8...3882afb 2024-04-16 [email protected] Fix leak memory in `Tooltip` and account detail (flutter/flutter#146833) 2024-04-16 [email protected] Fix memory leak in `BottomNavigationBar` (flutter/flutter#146748) 2024-04-16 [email protected] Roll Flutter Engine from cba7678fcfd2 to 71ab8854f14e (2 revisions) (flutter/flutter#146819) 2024-04-16 [email protected] Roll Flutter Engine from 66ad80217898 to cba7678fcfd2 (6 revisions) (flutter/flutter#146811) 2024-04-16 [email protected] Roll pub packages (flutter/flutter#146797) 2024-04-16 [email protected] Roll Flutter Engine from 76fbe31a96b3 to 66ad80217898 (2 revisions) (flutter/flutter#146794) 2024-04-16 [email protected] Roll Flutter Engine from 557ffb1cfda6 to 76fbe31a96b3 (4 revisions) (flutter/flutter#146792) 2024-04-16 [email protected] Roll Flutter Engine from 503e7e8f37f3 to 557ffb1cfda6 (1 revision) (flutter/flutter#146791) 2024-04-16 [email protected] Roll Flutter Engine from 07ae93c9b755 to 503e7e8f37f3 (7 revisions) (flutter/flutter#146790) 2024-04-15 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 3.24.10 to 3.25.0 (flutter/flutter#146784) 2024-04-15 [email protected] Roll pub packages (flutter/flutter#146782) 2024-04-15 [email protected] Roll Flutter Engine from 84238c411001 to 07ae93c9b755 (3 revisions) (flutter/flutter#146776) 2024-04-15 [email protected] Add hitTestBehavior and rootOverlay to LongPressDraggable constructor� (flutter/flutter#146386) 2024-04-15 [email protected] Roll Flutter Engine from 1a13c7d1f40e to 84238c411001 (4 revisions) (flutter/flutter#146775) 2024-04-15 [email protected] Update leak related TODOs. (flutter/flutter#146743) 2024-04-15 [email protected] [Doctor] Improve CocoaPods messages (flutter/flutter#146701) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
flutter#146386) Passes through these two Draggable parameters so that they're usable in LongPressDraggable.
) Manual roll requested by [email protected] flutter/flutter@2e748e8...3882afb 2024-04-16 [email protected] Fix leak memory in `Tooltip` and account detail (flutter/flutter#146833) 2024-04-16 [email protected] Fix memory leak in `BottomNavigationBar` (flutter/flutter#146748) 2024-04-16 [email protected] Roll Flutter Engine from cba7678fcfd2 to 71ab8854f14e (2 revisions) (flutter/flutter#146819) 2024-04-16 [email protected] Roll Flutter Engine from 66ad80217898 to cba7678fcfd2 (6 revisions) (flutter/flutter#146811) 2024-04-16 [email protected] Roll pub packages (flutter/flutter#146797) 2024-04-16 [email protected] Roll Flutter Engine from 76fbe31a96b3 to 66ad80217898 (2 revisions) (flutter/flutter#146794) 2024-04-16 [email protected] Roll Flutter Engine from 557ffb1cfda6 to 76fbe31a96b3 (4 revisions) (flutter/flutter#146792) 2024-04-16 [email protected] Roll Flutter Engine from 503e7e8f37f3 to 557ffb1cfda6 (1 revision) (flutter/flutter#146791) 2024-04-16 [email protected] Roll Flutter Engine from 07ae93c9b755 to 503e7e8f37f3 (7 revisions) (flutter/flutter#146790) 2024-04-15 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 3.24.10 to 3.25.0 (flutter/flutter#146784) 2024-04-15 [email protected] Roll pub packages (flutter/flutter#146782) 2024-04-15 [email protected] Roll Flutter Engine from 84238c411001 to 07ae93c9b755 (3 revisions) (flutter/flutter#146776) 2024-04-15 [email protected] Add hitTestBehavior and rootOverlay to LongPressDraggable constructor� (flutter/flutter#146386) 2024-04-15 [email protected] Roll Flutter Engine from 1a13c7d1f40e to 84238c411001 (4 revisions) (flutter/flutter#146775) 2024-04-15 [email protected] Update leak related TODOs. (flutter/flutter#146743) 2024-04-15 [email protected] [Doctor] Improve CocoaPods messages (flutter/flutter#146701) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Pre-launch Checklist
///).