-
Notifications
You must be signed in to change notification settings - Fork 5
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
Geotagging improvements #15
Comments
@supranove can you add a satellite imagery layer to the map? |
@elliec sure thing. |
In addition to code improvements, the following features have been added: - Zoom slider in addition to the zoom buttons. - Full-screen mode. - Aerial and satellite imagery. References #15.
It is now possible to set a map's default location from the project configuration file. References #15.
@elliec I was wondering, is the tour (for a user's first interaction with the geotagging map) necessary now that we have a help modal? And if it is, what should I put in it that won't be in the help? |
@supranove you can leave out the tour for now, we can see what we get in feedback. I will leave this open while we test it but looks good, once testing goes well can close thanks! |
The search functionality previously centered the map at a location's longitude and latitude. The zoom factor never changed which became problematic when users searched for very specific locations, e.g. streets or building addresses, because while the map was correctly centered at the location, it did not automatically zoom in to provide more detail. The search functionality now centers the map around the location's bounding box, i.e. a rectangle within which the location is encompassed, effectively using a variable zoom factor based on the bounding box's area. References #15.
Add the following
search location box(see issue Fix location search function in geotagging questions. #8).The text was updated successfully, but these errors were encountered: