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

UX: getting local (set default zoom, geolocate map, and add control scale) #14

Open
kidwellj opened this issue Jul 10, 2017 · 8 comments
Assignees

Comments

@kidwellj
Copy link
Member

We should set default zoom to something a bit closer to local area for a new load of the map (level 9 maybe?); add some code to geolocate the current user and centre the map on their location; and include display with a scale (see: http://leafletjs.com/reference-1.1.0.html#control-scale).

@kidwellj
Copy link
Member Author

kidwellj commented Jul 10, 2017

Just added commit (master ce1cab5) to set zoom @ 9.

@kidwellj
Copy link
Member Author

kidwellj commented Jul 10, 2017

Just added commit (c3d7746) to include scale (with imperial!).

@kidwellj
Copy link
Member Author

Looks like this is the ticket for user geolocation: https://github.com/domoritz/leaflet-locatecontrol. But I think we'll want to leave this code commented out for now as it will be annoying for testing until we get more data in England/Wales

@kidwellj
Copy link
Member Author

Re-opening so that we can remember to add scale to display (in "imperial"):
i.e. L.control.scale().addTo(map);

@kidwellj kidwellj reopened this Aug 31, 2017
@danhillcode
Copy link
Collaborator

where did you see this going as at the moment it gets placed underneath the left sidebar by default

@kidwellj
Copy link
Member Author

kidwellj commented Sep 10, 2017

Can we centre it? LH Sidebar covers it otherwise when it's up. Or add behaviour so that it slides to the right when LH Sidebar overlaps?

@danhillcode
Copy link
Collaborator

I have entered it looks decent now.

@danhillcode
Copy link
Collaborator

also increased zoom as requested 327-334 are the lines that control this in the index.html file

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

No branches or pull requests

2 participants