[openstreetmap/openstreetmap-website] Use Turbo frames for sidebar loading (PR #7245)

Marwin Hochfelsner notifications at github.com
Wed Jul 22 13:19:39 UTC 2026


@hlfan commented on this pull request.



> @@ -263,7 +263,7 @@ def map_layout
 
     flash.now[:warning] = { :partial => "layouts/offline_flash" } unless api_status == "online"
 
-    request.xhr? ? "xhr" : "map"
+    request.headers["turbo-frame"] == "sidebar_content_frame" ? "xhr" : "map"

The one concern I have about that is how to provide the atom links in that case, if they are still necessary.

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

Message ID: <openstreetmap/openstreetmap-website/pull/7245/review/4754720021 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260722/5957a85c/attachment.htm>


More information about the rails-dev mailing list