-
Notifications
You must be signed in to change notification settings - Fork 12
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
Get the leaflet map working with CAs #10
Comments
I asked Land Survey students to get some idea about the coordinate system. I used another approach to generate the geojson. Please check https://github.com/2blam/HK-geojson The map can show constituency area in Hong Kong. When the mouse clicked on the polygon, it shows the corresponding CA names. @vincentlaucy see if it can help and put it to frontend |
Thanks @2blam, Example for ref: Ideally IMO we can reuse the same map, when click to DC we increase zoom On Tue, Jan 28, 2014 at 7:01 PM, 2blam notifications@github.com wrote:
|
Sounds like a good strategy to me. So to summarize:
However, once zoomed into the CA, I would say that you need at least one links next to the map: a Zoom out or Back link to return to the DC view. The other link may be to Select all, if you wanted to select more than one CA, but it would actually be better if you could do this without zooming into the CA view at all. Also, I think any implementation should allow toggling of selecting more than one CA. In some cases, we only want to be able to select one CA. In other cases, we may want to select more than one. |
@vincentlaucy are you working on the maps tonight? If not, I will take a look at it in a bit. |
Not working at it at the moment. Can see anything to help on sun
|
OK, no worries, it's a holiday, people should be with their families :-). Just thinking out loud as I work through this: Taking a look at the geojson files, it looks like we still need to do a few things:
Next, to implement the actual interface, I will make a directive for the map. Going to plan it out with a spec and tests first, but it should probably implement the following features:
|
@2blam did a simplification inside qgis and got a smaller geojson
|
Ah okay, I remember now. Thanks for reminding me. On Fri, Jan 31, 2014 at 8:59 PM, vincentlaucy notifications@github.comwrote:
|
Fixed by pull request |
Partially implemented in
MapCtrl
The text was updated successfully, but these errors were encountered: