[openstreetmap-website] Port to Leaflet (#121)

Tom Hughes notifications at github.com
Sat Nov 3 15:26:33 GMT 2012


I haven't looked at the code yet, but a few comments based on a reviewing the site in action:

* Activating the "manually select an area" option in the export tab is very visually jarring because it triggers a zoom out in order to be able to highlight the whole of the current view as the selected area. I suspect it would be better to start with a smaller area selected that didn't trigger the zoom out.
* The user settings page doesn't show the location marker on the map - probably related to the `Couldn't autodetect L.Icon.Default.imagePath, set it manually` error that I see in the javascript console.
* Layout of popups on the map on the user page has changed - is this deliberate?
* The `bbox` parameter has vanished from the "view area on larger map" links on the data browser pages for ways etc, which means the box doesn't appear on the map when you follow it.
* The node view in the data browser reports `e is undefined` and the map never appears.

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/121#issuecomment-10040382
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20121103/ff27cd56/attachment.html>


More information about the rails-dev mailing list