#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/5...@github.com>
_______________________________________________ rails-dev mailing list rails-dev@openstreetmap.org https://lists.openstreetmap.org/listinfo/rails-dev
