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

Support for ground overlays #78

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Support for ground overlays #78

wants to merge 4 commits into from

Conversation

acq
Copy link
Contributor

@acq acq commented Jan 27, 2016

The first commit allows AirMapView to support Ground Overlays.
The two after rework the sample to allow multiple demos to coexist (through a navigation drawer) and add a demo for the newly added ground overlays

implements OnCameraChangeListener,
OnMapClickListener, OnCameraMoveListener, OnMapMarkerClickListener,
OnInfoWindowClickListener, OnLatLngScreenLocationCallback {
AirMapView mMap;
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: remove m prefix

@petzel
Copy link
Contributor

petzel commented Mar 4, 2016

This is great! Sorry for the delayed review - just a few minor things.

@acq
Copy link
Contributor Author

acq commented Apr 11, 2016

Hi
I finally integrated your feedback, and rebased on all the new things that happened in the last couple of months.
I also took the liberty of extracting the GeoJson demo from the menu to a new item in the navigation drawer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants