Skip to content

[google_maps_flutter_web] Fix AdvancedMarker anchors on web#11966

Open
3ph wants to merge 3 commits into
flutter:mainfrom
3ph:fixes-marker-anchor-on-web
Open

[google_maps_flutter_web] Fix AdvancedMarker anchors on web#11966
3ph wants to merge 3 commits into
flutter:mainfrom
3ph:fixes-marker-anchor-on-web

Conversation

@3ph

@3ph 3ph commented Jun 23, 2026

Copy link
Copy Markdown

Fixes AdvancedMarker anchor handling in google_maps_flutter_web.

Previously AdvancedMarker.anchor was not applied on web, so custom advanced marker content was positioned as if it used the default anchor. This caused markers with non-default anchors to appear offset from their intended map coordinates.

This PR converts Flutter marker anchor offsets into the Google Maps JavaScript Advanced Marker anchorLeft and anchorTop CSS offset properties when creating marker options, and reapplies those properties when an existing advanced marker is updated.

Adds integration test coverage for creating and updating an AdvancedMarker with custom anchors.

Addresses flutter/flutter#80578.

Pre-Review Checklist

@google-cla

google-cla Bot commented Jun 23, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request implements AdvancedMarker anchor handling for the Google Maps Flutter Web plugin. It introduces helper functions to convert anchor offsets to CSS percentage strings and apply them as anchorLeft and anchorTop properties on the underlying AdvancedMarkerElement and its options. Additionally, an integration test has been added to verify that anchors are correctly set and updated. I have no feedback to provide.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request fixes the anchor handling for AdvancedMarker on the web platform by introducing helper functions to convert anchor offsets to CSS percentages and apply them to AdvancedMarkerElementOptions and AdvancedMarkerElement. It also adds an integration test to verify anchor positioning and updates, and bumps the package version to 0.6.2+4. There are no review comments, and I have no feedback to provide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant