Skip to content
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

[google_maps_flutter_platform_interface] Add improvements to BitmapDescriptor API #6687

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jokerttu
Copy link
Contributor

@jokerttu jokerttu commented May 8, 2024

This PR adds improvements to BitmapDescriptor API for google_maps_flutter_platform_interface

  • Adds better support for marker size and scaling behaviour with AssetMapBitmap and BytesMapBitmap.
  • Deprecates BitmapDescriptor.fromAssetImage in favor of BitmapDescriptor.asset and AssetMapBitmap.create.
  • Deprecates BitmapDescriptor.fromBytes in favor of BitmapDescriptor.bytes and BytesMapBitmap

This is prequel PR for: #4055
Containing only changes to google_maps_flutter_platform_interface package.

Follow up PR:s will hold the platform and app-facing plugin implementations.

Linked issue: flutter/flutter#34657

Pre-launch Checklist

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

stuartmorgan

This comment was marked as resolved.

@jokerttu jokerttu force-pushed the feature/google_maps_flutter_platform_interface_bitmapdescriptor_improvements branch from 04e97b8 to c7a56e4 Compare May 13, 2024 10:45
@jokerttu jokerttu force-pushed the feature/google_maps_flutter_platform_interface_bitmapdescriptor_improvements branch 2 times, most recently from 937a3ee to 8fa54ff Compare May 16, 2024 07:28
- Improves public api for width and height values of the MapBitmap
- Also includes small fix to test naming.
@jokerttu jokerttu force-pushed the feature/google_maps_flutter_platform_interface_bitmapdescriptor_improvements branch from 8fa54ff to 2970bbd Compare May 20, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants