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

Missing possibility to use addresses #170

Open
GRAFIX21 opened this issue May 23, 2017 · 1 comment
Open

Missing possibility to use addresses #170

GRAFIX21 opened this issue May 23, 2017 · 1 comment

Comments

@GRAFIX21
Copy link

GRAFIX21 commented May 23, 2017

Hello, first I would like to say: Maplace feels really well constructed.

But I have one special need for less experienced users. Currently I mainly use the gomaps-plugin for displaying maps, because we can set markers by coordinates or simply by an address (where the script makes the work to get the geo-data from google maps). Sure this ist not the prefered way and even has a bad performance. But it is the most simple way for some people and for setting just a hand full of markers on websites with less traffic.

Could you implement a comparable solution to set markers not by a pair of latitude and longitute but by giving the postal address in the data array?

@danielemoraschi
Copy link
Owner

danielemoraschi commented May 23, 2017

Hi, thank you for the feedback!

I still think that Maplace should stay as "tiny" as possible and let the user implement the application logic using the library just as an helper to place markers on the map. It should not be Maplace responsibility handle how those locations get generated. Also, other users not interested in the geolocation functionality will have to load unused code in their pages for no reason.
But I see your point, and I will look to implement some sort of plugin mechanism to plug new functionalities only as necessary, or something similar.

For now, please check out my example with user input address and let me know if it's of any help: https://jsfiddle.net/_tomorro/Lfwn7g6f/
I'm sure it can be extended quite easily to manage more complex stuff.

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

2 participants