-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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 tarrinneal@google.com flutter/flutter@2e748e8...3882afb 2024-04-16 32538273+ValentinVignal@users.noreply.github.com Fix leak memory in `Tooltip` and account detail (flutter/flutter#146833) 2024-04-16 32538273+ValentinVignal@users.noreply.github.com Fix memory leak in `BottomNavigationBar` (flutter/flutter#146748) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from cba7678fcfd2 to 71ab8854f14e (2 revisions) (flutter/flutter#146819) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 66ad80217898 to cba7678fcfd2 (6 revisions) (flutter/flutter#146811) 2024-04-16 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#146797) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 76fbe31a96b3 to 66ad80217898 (2 revisions) (flutter/flutter#146794) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 557ffb1cfda6 to 76fbe31a96b3 (4 revisions) (flutter/flutter#146792) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 503e7e8f37f3 to 557ffb1cfda6 (1 revision) (flutter/flutter#146791) 2024-04-16 engine-flutter-autoroll@skia.org 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 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#146782) 2024-04-15 engine-flutter-autoroll@skia.org Roll Flutter Engine from 84238c411001 to 07ae93c9b755 (3 revisions) (flutter/flutter#146776) 2024-04-15 amirpanahandeh@yahoo.com Add hitTestBehavior and rootOverlay to LongPressDraggable constructor� (flutter/flutter#146386) 2024-04-15 engine-flutter-autoroll@skia.org Roll Flutter Engine from 1a13c7d1f40e to 84238c411001 (4 revisions) (flutter/flutter#146775) 2024-04-15 polinach@google.com Update leak related TODOs. (flutter/flutter#146743) 2024-04-15 737941+loic-sharma@users.noreply.github.com [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 rmistry@google.com,stuartmorgan@google.com,tarrinneal@google.com 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 tarrinneal@google.com flutter/flutter@2e748e8...3882afb 2024-04-16 32538273+ValentinVignal@users.noreply.github.com Fix leak memory in `Tooltip` and account detail (flutter/flutter#146833) 2024-04-16 32538273+ValentinVignal@users.noreply.github.com Fix memory leak in `BottomNavigationBar` (flutter/flutter#146748) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from cba7678fcfd2 to 71ab8854f14e (2 revisions) (flutter/flutter#146819) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 66ad80217898 to cba7678fcfd2 (6 revisions) (flutter/flutter#146811) 2024-04-16 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#146797) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 76fbe31a96b3 to 66ad80217898 (2 revisions) (flutter/flutter#146794) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 557ffb1cfda6 to 76fbe31a96b3 (4 revisions) (flutter/flutter#146792) 2024-04-16 engine-flutter-autoroll@skia.org Roll Flutter Engine from 503e7e8f37f3 to 557ffb1cfda6 (1 revision) (flutter/flutter#146791) 2024-04-16 engine-flutter-autoroll@skia.org 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 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#146782) 2024-04-15 engine-flutter-autoroll@skia.org Roll Flutter Engine from 84238c411001 to 07ae93c9b755 (3 revisions) (flutter/flutter#146776) 2024-04-15 amirpanahandeh@yahoo.com Add hitTestBehavior and rootOverlay to LongPressDraggable constructor� (flutter/flutter#146386) 2024-04-15 engine-flutter-autoroll@skia.org Roll Flutter Engine from 1a13c7d1f40e to 84238c411001 (4 revisions) (flutter/flutter#146775) 2024-04-15 polinach@google.com Update leak related TODOs. (flutter/flutter#146743) 2024-04-15 737941+loic-sharma@users.noreply.github.com [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 rmistry@google.com,stuartmorgan@google.com,tarrinneal@google.com 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
///).