Skip to content

Conversation

@Amir-P
Copy link
Contributor

@Amir-P Amir-P commented Apr 6, 2024

Pre-launch Checklist

@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Apr 6, 2024
@Amir-P Amir-P force-pushed the fix/long_press_draggable_missing_super_constructor_parameters branch from 351adbb to 429f4c8 Compare April 9, 2024 07:03
Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@justinmc justinmc left a 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!

@Amir-P Amir-P force-pushed the fix/long_press_draggable_missing_super_constructor_parameters branch from 429f4c8 to 9ebf730 Compare April 10, 2024 12:07
@Amir-P Amir-P requested a review from justinmc April 10, 2024 12:08
Copy link
Contributor

@justinmc justinmc left a 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 👍

@justinmc justinmc merged commit c98d68d into flutter:master Apr 15, 2024
@Amir-P Amir-P deleted the fix/long_press_draggable_missing_super_constructor_parameters branch April 16, 2024 05:07
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 16, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 16, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 16, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Apr 16, 2024
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
gilnobrega pushed a commit to gilnobrega/flutter that referenced this pull request Apr 22, 2024
flutter#146386)

Passes through these two Draggable parameters so that they're usable in LongPressDraggable.
TecHaxter pushed a commit to TecHaxter/flutter_packages that referenced this pull request May 22, 2024
)

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LongPressDraggable should have a flag to use rootOverlay and hitTestBehavior

3 participants