packages icon indicating copy to clipboard operation
packages copied to clipboard

[google_maps_flutter] Ground Overlay support for Android and iOS

Open gabbopalma opened this issue 1 year ago • 2 comments

This Pull Request should solve the lack of integration of Ground Overlays type objects within iOS and Android maps, on Flutter. Within the commits I worked on the Dart, Java and Objective-C code integrating the new geometry and aligning the code with the SDKs to which the device belongs (iOS or Android). In the past, this important type of geometry was not supported in any way by the Flutter package of Google Maps. I think it is essential to have GroundOverlays on Flutter as well, so I decided to work on it and attempt integration. The only gap of the current changes I have made is that the "fromPosition" constructor type does not work on iOS. But, in contrast, the "fromBounds" constructor works correctly. Constructive and helpful comments are welcome to improve the implementation and finally make GroundOverlay available to all developers who appreciate Google Maps.

Linked to 26479 issue.

Changes made:

  • google_maps_flutter:
    • Added methods related to GroundOverlays (such as updateGroundOverlays, onGroundOverlayTap etc.) and adapted existing code;
    • Added tests on GroundOverlays to the examples folder;
    • Pubspec.yaml and changelog updated.
  • google_maps_flutter_android:
    • Added GroundOverlays classes and its related code into the existing Java code;
    • Added test cases on GroundOverlays in the Java code;
    • Added the test cases on the GroundOverlays in the examples folder;
    • Pubspec.yaml and changelog updated.
  • google_maps_flutter_ios:
    • Added GroundOverlays classes and its related code into the existing Objective-C code;
    • Add tests on GroundOverlays in the examples folder;
    • Pubspec.yaml and changelog updated.
  • google_maps_flutter_platform_interface:
    • Added classes and methods related to GroundOverlays;
    • Added tests on GroundOverlays to the test folder;
    • Pubspec.yaml and changelog updated.

Pre-launch Checklist

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

gabbopalma avatar Jul 03 '24 15:07 gabbopalma

(triage) @gabbopalma Do you still have plans to finish this PR and take it out of draft mode?

goderbauer avatar Sep 10 '24 22:09 goderbauer

I will work on this on the next weeks. Actually, I have just to align the edits to the latest commits.

gabbopalma avatar Sep 10 '24 22:09 gabbopalma

Closing this as inactive, feel free to reopen once you have time to come back to this pr

chunhtai avatar Nov 12 '24 23:11 chunhtai