[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