[openstreetmap/openstreetmap-website] Add a browse map link to notes (#2724)
Andy Allan
notifications at github.com
Wed Jul 22 12:21:23 UTC 2020
This matches the one for nodes. Fixes #1355. Replacement for #1773.
An unstyled list works better for the details section, particularly when all three are shown.
The change to the named path in the nodes template is based on a review I gave in #1773.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/2724
-- Commit Summary --
* Use a named path instead of explicit controllers and actions
* Add a browse map link to notes
-- File Changes --
M app/views/browse/_common_details.html.erb (2)
M app/views/browse/note.html.erb (14)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/2724.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2724.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2724
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20200722/397aa5e9/attachment.htm>
More information about the rails-dev
mailing list