> Do we need another set of icons in addition to
> `app/assets/images/social_icons`?
They have different design. They are more colorful and more like logos which
should gain users' attention (for example as share buttons), but icons added by
this PR are one color, simple icons, which should not gain much attention as
they are there only for the information for the small part of the profile page
(not big functional buttons). We can use icons from the
`app/assets/images/social_icons`, but it will make `social profile links` too
colorful and too much attention grabbing. Therefore, I think, it is better to
stick to the new icons.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5439#issuecomment-2577314212
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5439/c2577314...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
