[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