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

Various changes on my fork, which you might like to include? #23

Open
johan12345 opened this issue Aug 8, 2020 · 0 comments
Open

Various changes on my fork, which you might like to include? #23

johan12345 opened this issue Aug 8, 2020 · 0 comments

Comments

@johan12345
Copy link

johan12345 commented Aug 8, 2020

Hi! Thank you very much for the AnyMaps library. I am working on implementing it into my EVMap project (https://github.com/johan12345/EVMap).
In the process have done some improvements to AnyMaps in my fork, as seen in the list below.

Is this project still maintained? Would you be open to PRs for some or all of these changes?

If yes, how should we handle this - one small PR for each of the changes? In that case it may be necessary to have the PRs built on top of each other as not all changes are completely separable from each other. Or I can make one large PR with the features you would like to have.

Major changes

  • Add a MapFragment class (ev-map@830989e, ev-map@e172821), which handles lifecycle callbacks automatically and allows the user to directly set which map implementation should be used. This alleviates the need to import a specific MapView and MapsConfiguration implementation and can automatically select the best backend based on the anymap-... libraries that are available (e.g. if you have one Build flavor for Google Maps and one for OSM, you don't need any flavor-specific code to handle the creation of the map).
  • Add an implementation for Mapbox maps in the anymaps-mapbox module (ev-map@61c95e6, ev-map@ebc4582)
  • Update dependencies (ev-map@1a3b385, ev-map@3d229c8, ev-map@5d14324) and make necessary code adjustments (Android build tools, Support Library, Google Maps, OSMDroid)

Minor changes (relevant for all AnyMaps backends)

Minor changes (OSM)

  • add attribution (ev-map@2a2c032)
  • Disable OSMDroid's zoom controller buttons (ev-map@8613bc3) for consistency with Google Maps, which does not have such buttons by default

Minor changes (Google Maps)

@johan12345 johan12345 changed the title Various changes on my fork, which would you like to include? Various changes on my fork, which you might like to include? Aug 8, 2020
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

No branches or pull requests

1 participant