@tomhughes commented on this pull request.
> @@ -0,0 +1,23 @@
+module OpenStreetMap
+ class SvgCurrentColorTransform < InlineSvg::CustomTransformation
+ def transform(doc)
+ doc.xpath("//@fill|//@stroke").each do |attr|
+ attr.value = "currentColor" if attr.value.strip.downcase == value
+ end
+ doc
+ end
+ end
+
+ class SvgToSymbolTransform < InlineSvg::CustomTransformation
+ def transform(doc)
+ with_svg(doc) do |svg|
+ svg.name = "symbol"
I don't think this transform is working - functionally everything does work but
if you look at the source the outer `svg` contains a bunch of `svg` elements
not `symbol` elements.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6189#pullrequestreview-3014108738
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6189/review/3014108...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
