android-maps-utils icon indicating copy to clipboard operation
android-maps-utils copied to clipboard

feat!: Add support for setOnInfoWindowCloseListener() for Markers

Open bshi opened this issue 5 months ago • 0 comments

BREAKING CHANGE: ClusterRenderer adds two InfoWindowClose listeners. Applications that used ClusterRenderer or MarkerManager with GoogleMap.setOnInfoWindowCloseListener must now register their listener(s) with relevant collection/clustering management class.

Circa 2020, we added some more InfoWindow-related events to Marker/ClusterManager but InfoWindowClose was not included. This change addresses a discrepancy in the class documentation for MarkerManager which claims to "[d]elegates all Marker-related events to each ...". The close handler has been missing.

[1] https://github.com/googlemaps/android-maps-utils/commit/8a3fc0195a61cb794c01ca266f59f1c034487484

Addresses #1540


Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • [X] Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • [X] Edit the title of this pull request with a semantic commit prefix (e.g. "fix: "), which is necessary for automated release workflows to decide whether to generate a new release and what type it should be.
  • [X] Will this cause breaking changes to existing Java or Kotlin integrations? If so, ensure the commit has a BREAKING CHANGE footer so when this change is integrated a major version update is triggered. See: https://www.conventionalcommits.org/en/v1.0.0/
  • [ ] Ensure the tests and linter pass
  • [ ] Code coverage does not decrease (if any source code was changed)
  • [X] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

bshi avatar Jul 11 '25 17:07 bshi