mmd-osm left a comment (openstreetmap/openstreetmap-website#5880)
For additional context: the "icon" in line 137 of `external_svg` refers to an
additional `id="icon"` I had to add to each of the external svgs, like in this
example:
```xml
<svg xmlns="http://www.w3.org/2000/svg"; id="icon" width="12" height="12"
viewBox="0 0 12 12">
```
I haven't really tested, if it's possible to externalize `fill="currentColor"`
by adding a css class (e.g. education) to the svg:
```
.education {
fill: #39ac39;
}
```
... and then use "currentColor" instead of "#39ac39".
External svgs (rather than inlining) would benefit from browser caching.
Downloading lots of small SVGs shouldn't be much of an issue with HTTP/2.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/5880#issuecomment-2784431604
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/issues/5880/2784431...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
