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

Project Submit #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Project Submit #44

wants to merge 1 commit into from

Conversation

tdrier1
Copy link

@tdrier1 tdrier1 commented Jun 28, 2017

I have made several updates:

Added text search functionality with jQuery autocomplete
Added custom pins with css formatted info windows
Added ability to delete a marker with a right click
Moved layout around for a better user experience

Since I am new to Javascript there a few interesting things I noticed. The first of which was learning how to work with the google maps API. This was actually very user friendly and their documentation is extremely good. The other would be how the data flows from Js to HTML, I am used to using a getElementbyID, so it is always interesting to see different ways of accomplishing the same thing.

My suggestion for improvement would be add a collection of current pins boxes the user can click on and have the map snap to that location when clicked. I feel that would help improve the quality and usefulness of this app.

I feel like I went above and beyond with formatting the info windows. This really helps make the information easier to read and helps with the overall design flow.

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.

1 participant