@AntonKhorev commented on this pull request.
> </small>
+ <a href="#c<%= comment.id %>">
+ <div class="icon link"></div>
+ </a>
Here's what I'd do:
```suggestion
<a href="#c<%= comment.id %>">
<svg width="16" height="16" fill="currentColor">
<path d="M4.715 6.542 3.343 7.914a3 3 0 1 0 4.243
4.243l1.828-1.829A3 3 0 0 0 8.586 5.5L8 6.086a1 1 0 0 0-.154.199 2 2 0 0 1 .861
3.337L6.88 11.45a2 2 0 1 1-2.83-2.83l.793-.792a4 4 0 0 1-.128-1.287z"/>
<path d="M6.586 4.672A3 3 0 0 0 7.414 9.5l.775-.776a2 2 0 0
1-.896-3.346L9.12 3.55a2 2 0 1 1 2.83 2.83l-.793.792c.112.42.155.855.128
1.287l1.372-1.372a3 3 0 1 0-4.243-4.243z"/>
</svg>
</a>
</small>
```
This is [the rotated icon](https://icons.getbootstrap.com/icons/link-45deg/).
Yes it's going to be repeated as many times as there are comments, but that
shouldn't be a big problem with compression. The advantage to inlining is that
you can use `currentColor` instead of specifying the color.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4789#pullrequestreview-2660446072
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/4789/review/2660446...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
