@hlfan commented on this pull request.
> +<defs>
+ <path id="icon-note" fill="currentColor" d="M17
12H15v2H13v2h2v2h2V16h2V14H17V12ZM3 12H6l2 3 2-3h3l1-1V4L13 3H3L2 4v7Z" />
+</defs>
I didn't particularly like having the same path written two different ways
(with and without tag helpers) in three different files either. I think it
makes `welcome` and `fixthemap` a bit easier to read but I can remove the
template if wanted.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5752#discussion_r2021995324
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5752/review/2730865...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
