[openstreetmap/openstreetmap-website] Clean up some svg-related markup (PR #6239)

Marwin Hochfelsner notifications at github.com
Thu Jul 24 11:40:54 UTC 2025


Two things:

- A thing I missed in #6200 that was made possible by not needing to sometimes render the definition.
- A thing where the 13 minutes between the merges of #6215 and #5854 weren't enough to adapt the code to the test-drive.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/6239

-- Commit Summary --

  * Move svg wrapper into control_icons partial
  * Move marker colors to colors.scss

-- File Changes --

    M app/assets/stylesheets/colors.scss (6)
    M app/assets/stylesheets/common.scss (3)
    M app/assets/stylesheets/print.scss (3)
    M app/views/dashboards/show.html.erb (2)
    M app/views/diary_entries/_form.html.erb (2)
    M app/views/layouts/_control_icons.html.erb (8)
    M app/views/layouts/map.html.erb (2)
    M app/views/profiles/locations/show.html.erb (2)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/6239.patch
https://github.com/openstreetmap/openstreetmap-website/pull/6239.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/6239 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250724/d7e4aa17/attachment.htm>


More information about the rails-dev mailing list