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

migration to gl js #68

Closed
wants to merge 1 commit into from
Closed

migration to gl js #68

wants to merge 1 commit into from

Conversation

hrecht
Copy link
Contributor

@hrecht hrecht commented Aug 5, 2016

I worked on #58 for a bit at Code for DC - this branch migrates the map to mapox-gl.js and draws the bike lanes.

@alulsh
Copy link
Member

alulsh commented Aug 11, 2016

Thanks for getting this started @hrecht! Just granted you write access to the repo - mind pushing this directly as a feature branch in another PR instead of a PR from you fork? Will make the workflow much easier.

A couple of things to fix in this PR before we can merge:

  • Fix the right hand corner zoom control - at the moment the div at the top covers it and you can only zoom out, not zoom in
  • Fix the buffers

Once this is PR'd as a work in progress (WIP) PR directly to this repo, others can add commits to the branch and tackle the above tasks.

Thanks again for working on this @hrecht!

@hrecht hrecht closed this Aug 20, 2016
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