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

Refactor LocationSimulation: Replace Google Maps with Open Street Map #3863

Merged
merged 2 commits into from
Jan 22, 2025

Conversation

SamC1832js
Copy link
Contributor

Refactor LocationSimulation: Replace Google Maps with OpenStreetMap

  • Replaced the Google Maps implementation with OpenStreetMap in the LocationSimulation feature.

  • Enhanced the functionality to dynamically update form fields (latitude and longitude) when the map position changes.

  • Known Issue: When LocationSimulation is closed and reopened, the map does not display. The current workaround is to keep LocationSimulation open throughout the development process.

Refactor LocationSimulation: Replace Google Maps with OpenStreetMap

- Replaced the Google Maps implementation with OpenStreetMap in the LocationSimulation feature.
- Enhanced the functionality to dynamically update form fields (latitude and longitude) when the map position changes.

- Known Issue: When LocationSimulation is closed and reopened, the map does not display. The current workaround is to keep LocationSimulation open throughout the development process.
@SamC1832js
Copy link
Contributor Author

Refactor LocationSimulation: Replace Google Maps with OpenStreetMap
Resolves issue #3173.

  • Replaced Google Maps with OpenStreetMap for LocationSimulation.
  • Tested functionality using a bare-bones Java app via Codename One Initializr.
  • Enhanced the map to dynamically update the form fields (latitude and longitude) as the map position changes.

Known Issue:

  • When LocationSimulation is closed and reopened, the map does not display.
  • Workaround: Keep LocationSimulation open during the development process.

@shai-almog
Copy link
Collaborator

Thanks!
This looks great. Just a couple of comments to get the CI fixed and it can be merged: https://github.com/codenameone/CodenameOne/actions/runs/12897154590/job/35968885875?pr=3863

…oogle map page and change lambdas to anonymous classes to follow Java 7 syntax.

Tested functionality using a bare-bones Java app via Codename One Initializr.

pull request: codenameone#3863
issue: codenameone#3173

Known Issue:
- When LocationSimulation is closed and reopened, the map does not display.
-Workaround: Keep LocationSimulation open during the development process.
@SamC1832js
Copy link
Contributor Author

I have committed changes to remove the commented Google map page and changed lambdas to anonymous classes to comply with Java 7 syntax. 3863

@shai-almog shai-almog merged commit b449bde into codenameone:master Jan 22, 2025
1 check passed
@shai-almog
Copy link
Collaborator

Great job, thanks!

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