[openstreetmap/openstreetmap-website] Deduplicate social link icon assets (PR #6787)
Marwin Hochfelsner
notifications at github.com
Tue Feb 10 03:04:51 UTC 2026
There's no need to store the OSM logo four times with basically no difference.
|before|after|
|-|-|
|<img src="https://github.com/user-attachments/assets/8ba98770-d973-4505-9843-81bf53d9d2c2" />|<img src="https://github.com/user-attachments/assets/e131aa6c-cbf4-4552-9965-865ac2f90c1b" />|
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/6787
-- Commit Summary --
* Add asset routing transformation for inline SVG images
* Reuse osm logo in related social link icons
* Collect chat bubble transforms
-- File Changes --
M app/assets/images/social_link_icons/ohm.svg (235)
M app/assets/images/social_link_icons/osm_forum.svg (694)
M app/assets/images/social_link_icons/osm_wiki.svg (339)
M config/initializers/inline_svg.rb (11)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/6787.patch
https://github.com/openstreetmap/openstreetmap-website/pull/6787.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6787
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6787 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260209/231f898f/attachment-0001.htm>
More information about the rails-dev
mailing list