Skip to content

Conversation

@jokerttu
Copy link
Contributor

@jokerttu jokerttu commented Sep 14, 2024

Adds ability to configure camera animation duration on Android and iOS.

Resolves #39810
Resolves #44284

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch from fbf20bc to 68e52df Compare September 14, 2024 12:57
@jokerttu jokerttu changed the title [google_maps_flutter] add ability to animate camera with duration [google_maps_flutter] Add ability to animate camera with duration Sep 14, 2024
@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch from 68e52df to 2c8caf1 Compare September 15, 2024 09:03
@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch 2 times, most recently from 2256a07 to c39c721 Compare November 7, 2024 16:41
@jokerttu jokerttu marked this pull request as ready for review November 7, 2024 16:58
@stuartmorgan-g stuartmorgan-g added the federated: all_changes PR that contains changes for all packages for a federated plugin change label Nov 7, 2024
@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch from 7c30a33 to 167b519 Compare November 11, 2024 14:13
@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch from 167b519 to 6078134 Compare November 21, 2024 11:42
@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch 2 times, most recently from eb71ec7 to b6807ec Compare November 22, 2024 08:24
@stuartmorgan-g stuartmorgan-g added triage-ios Should be looked at in iOS triage triage-android Should be looked at in Android triage labels Nov 22, 2024
@stuartmorgan-g
Copy link
Collaborator

My comments are all minor, so once the iOS and Android reviewers have signed off this will be ready to split out the first sub-PR.

@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch from b6807ec to 6bc7717 Compare November 25, 2024 08:00
@jesswrd jesswrd requested review from a team and jesswrd November 26, 2024 19:34
@reidbaker reidbaker requested a review from camsim99 December 3, 2024 19:22
@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch from 261d5b0 to 028c147 Compare March 20, 2025 09:54
@jokerttu jokerttu force-pushed the feature/animate-camera-duration branch from 028c147 to 32c21af Compare March 20, 2025 10:12
auto-submit bot pushed a commit that referenced this pull request Mar 24, 2025
…latform impls (#8659)

This PR contains platform implementations for the ability to animate camera with duration (#7648).

Linked issue: flutter/flutter#39810
Linked issue: flutter/flutter#44284
@flutter-dashboard

This comment was marked as resolved.

@jokerttu
Copy link
Contributor Author

@stuartmorgan this PR now contains only app-facing package changes

@jokerttu jokerttu requested a review from stuartmorgan-g March 24, 2025 12:01
Copy link
Collaborator

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

LGTM!

@jokerttu jokerttu added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 25, 2025
@auto-submit auto-submit bot merged commit 125c117 into flutter:main Mar 25, 2025
82 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 25, 2025
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Mar 25, 2025
flutter/packages@c27d2fe...125c117

2025-03-25 joonas.kerttula@codemate.com [google_maps_flutter] Add
ability to animate camera with duration (flutter/packages#7648)
2025-03-24 15619084+vashworth@users.noreply.github.com [webview_flutter]
Fix macOS test issues and skip some legacy iOS tests
(flutter/packages#8895)

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-flutter-autoroll
Please CC flutter-ecosystem@google.com on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
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
dko5ki23t pushed a commit to dko5ki23t/google_maps_flutter_improved that referenced this pull request May 24, 2025
…n (#8596)

This PR contains platform interface changes for support to adjust camera animation speed (flutter/packages#7648).

Linked issue:  flutter/flutter#39810
Linked issue:  flutter/flutter#44284
androidseb pushed a commit to androidseb/packages that referenced this pull request Jun 8, 2025
flutter#8596)

This PR contains platform interface changes for support to adjust camera animation speed (flutter#7648).

Linked issue:  flutter/flutter#39810
Linked issue:  flutter/flutter#44284
androidseb pushed a commit to androidseb/packages that referenced this pull request Jun 8, 2025
…latform impls (flutter#8659)

This PR contains platform implementations for the ability to animate camera with duration (flutter#7648).

Linked issue: flutter/flutter#39810
Linked issue: flutter/flutter#44284
androidseb pushed a commit to androidseb/packages that referenced this pull request Jun 8, 2025
…utter#7648)

Adds ability to configure camera animation duration on Android and iOS.

Resolves [#39810](flutter/flutter#39810)
Resolves [#44284](flutter/flutter#44284)
zhangyuang pushed a commit to zhangyuang/flutter-fork that referenced this pull request Jun 9, 2025
…r#165884)

flutter/packages@c27d2fe...125c117

2025-03-25 joonas.kerttula@codemate.com [google_maps_flutter] Add
ability to animate camera with duration (flutter/packages#7648)
2025-03-24 15619084+vashworth@users.noreply.github.com [webview_flutter]
Fix macOS test issues and skip some legacy iOS tests
(flutter/packages#8895)

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-flutter-autoroll
Please CC flutter-ecosystem@google.com on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
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
FMorschel pushed a commit to FMorschel/packages that referenced this pull request Jun 9, 2025
flutter#8596)

This PR contains platform interface changes for support to adjust camera animation speed (flutter#7648).

Linked issue:  flutter/flutter#39810
Linked issue:  flutter/flutter#44284
FMorschel pushed a commit to FMorschel/packages that referenced this pull request Jun 9, 2025
…latform impls (flutter#8659)

This PR contains platform implementations for the ability to animate camera with duration (flutter#7648).

Linked issue: flutter/flutter#39810
Linked issue: flutter/flutter#44284
FMorschel pushed a commit to FMorschel/packages that referenced this pull request Jun 9, 2025
…utter#7648)

Adds ability to configure camera animation duration on Android and iOS.

Resolves [#39810](flutter/flutter#39810)
Resolves [#44284](flutter/flutter#44284)
romanejaquez pushed a commit to romanejaquez/flutter that referenced this pull request Aug 14, 2025
…r#165884)

flutter/packages@c27d2fe...125c117

2025-03-25 joonas.kerttula@codemate.com [google_maps_flutter] Add
ability to animate camera with duration (flutter/packages#7648)
2025-03-24 15619084+vashworth@users.noreply.github.com [webview_flutter]
Fix macOS test issues and skip some legacy iOS tests
(flutter/packages#8895)

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-flutter-autoroll
Please CC flutter-ecosystem@google.com on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
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

autosubmit Merge PR when tree becomes green via auto submit App federated: all_changes PR that contains changes for all packages for a federated plugin change p: google_maps_flutter triage-ios Should be looked at in iOS triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[google_maps_flutter] Zoom animations are extremely fast on iOS [google_maps_flutter] Change camera animation speed (and curve?)

6 participants