[openstreetmap/openstreetmap-website] Store routing icons separately (PR #6189)

[Tom Hughes]

@tomhughes commented on this pull request.



> @@ -0,0 +1,7 @@
+require Rails.root.join("lib/svg_to_symbol_transform.rb")
+require Rails.root.join("lib/replace_magenta_svg_color.rb")

I'd probably just put the custom transform classes inline here rather than putting them in `lib` especially as autoloading doesn't even work here - at least I assume that's why you had to require them explicitly.

Other initialisers that do similar things just define the code they want in place. I do usually put it in `module OpenStreetMap` though, to make it clear it's a customisation if it appears in a stack trace.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6189/review/3014038895 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250713/c8b13aa6/attachment.htm>