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

Map-demo #49

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kranthikumar050907
Copy link

updates made to the code:
1)coded lines to display complete information about the airports
2)sorted the drop-down to make it easy for the users to find the codes
3)added marker remove functionality to remove marker by clicking on it
4) centered the airport marker every time user clicks a new airport code
5)adjusted the map and the info box next to each other to in order to use less window space and easily visible for the users.

interesting things you noticed about the code: I never worked on google maps API, I got a chance to do it with this project and I learned that the map API provides a lot of flexibility to the programmer to make any changes.

suggestions for improvements: I can see the UI layout is pretty basic it can be enhanced by using basic bootstrap classes and any framework or use of libraries will also help

updated the map api with its key to allow the browser to render the
google map, and also added some code changes to Map.html for fsname and
some extra code lines to Map.js
added click event listener and setting the marker to null to remove the
marker where user clicked on
sorting the airport list in ascending order
centering the map to the most recently marked ariport code, helping use
to find the aiport easly
added new css attributes to get a new look for the map and list
@kranthikumar050907 kranthikumar050907 changed the title message-demo Map-demo Aug 10, 2017
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