[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