@AntonKhorev commented on this pull request.
>
- map.setSidebarOverlaid(!endpoints[0].latlng || !endpoints[1].latlng);
- }
+ map.setSidebarOverlaid(!endpoints[0].latlng || !endpoints[1].latlng);
Yes, it's still needed.
1. Reload the map view to make sure that the directions sidebar is not loaded.
2. Open the context menu.
3. Pick *Directions from here*.
4. If you removed `map.setSidebarOverlaid` from `page.pushstate` /
`page.popstate`, you'll see an empty *Directions* sidebar under the directions
form. Normally it doesn't appear until two endopints are picked.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5961#discussion_r2064856924
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5961/review/2801093...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev