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

Tom Hughes notifications at github.com
Wed Jul 22 17:48:00 UTC 2026


@tomhughes 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 changeset history view does generate atom links so I think that would be affected, but it we replaced the current `xhr` layout with our own version of `turbo_rails/frame` then we could include the auto_discovery_link_tag stuff?

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

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


More information about the rails-dev mailing list