@hlfan commented on this pull request.
> +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
Some icons in this repo (like
[ohm.svg](https://github.com/openstreetmap/openstreetmap-website/blame/master/app/assets/images/social_link_icons/ohm.svg))
say they were exported from Illustrator.
Illustrator has currently [no
support](https://illustrator.uservoice.com/forums/601447-illustrator-desktop-bugs/suggestions/42330448-please-allow-fill-currentcolor-to-be-set-for-svg)
for currentColor.
But I'm up for removing the transform if this will be checked in the PRs.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6189#discussion_r2203490637
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6189/review/3014279...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
