[openstreetmap/openstreetmap-website] Remove controls sprite (PR #5752)
Marwin Hochfelsner
notifications at github.com
Tue Apr 1 01:37:07 UTC 2025
@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/2730865090 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250331/9088ef3c/attachment.htm>
More information about the rails-dev
mailing list