[openstreetmap/openstreetmap-website] Unify directions page handler and parallelize requests (PR #5967)

Marwin Hochfelsner notifications at github.com
Wed Apr 30 08:13:33 UTC 2025


#5961's fix to #5959 was to delay starting the route request until the sidebar loaded, which isn't needed.

With this fix to #5959, I'm merging the page's pushstate/popstate with load and pulling out the sidebar requirement into a helper function also used in getRoute after the request, enabling the requests to be run in parallel.

This should remove half a second of extra latency for people with geostationary satellite internet.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5967

-- Commit Summary --

  * Unify directions page handler and parallelize requests

-- File Changes --

    M app/assets/javascripts/index/directions.js (48)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5967.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5967.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5967
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5967 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250430/6d69945a/attachment-0001.htm>


More information about the rails-dev mailing list