@hlfan 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"

Given there isn't a "lib" to test against anymore, a simple DOM check seems the 
most straightforward to me.
Or should testing that be done somewhere else?

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/6189/review/3014123...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to