Skip to content

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