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

Issue#52 #79

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

Issue#52 #79

wants to merge 7 commits into from

Conversation

LittleBaiBai
Copy link

This fixes issues #51 and #52. Review without whitespace changes.

@carpeliam
Copy link
Member

carpeliam commented Oct 30, 2016

Hm, is this the expected behavior when you click on multiple locations on the map? Seems a little tricky...

cambridge_voting_locations

To reproduce, I set my starting location to 145 Broadway and then clicked on the markers on the map. Clicking on any marker added directions, and clicking that marker twice removed the directions to that location, but clicking on other additional markers did not clear the original directions.

Pushing additional changes to the same feature branch will also get added to this PR, so you can make changes if you'd like. Otherwise if you don't think you can look into it by Tuesday, let me know and I can try to take a pass at it.

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