@markup-mitchell
Posted
- use viewbox. eg your social media icons: remove
width="20"
andheight="20"
and useviewbox="0 0 20 20"
instead. Now use CSS to setwidth
/height
in whatever units you like. - images are displayed as
inline
elements by default and the gap at the bottom is for text descenders. Usedisplay: block
orvertical-align: middle
on yourimg
elements.
I didn't look at your semantics and accessibility in much detail; I'm suspicious of aria tags as they're often unnecessary and can make things worse if misused, but hiding the SVG and labelling the link seems like a safe bet to me. Good effort!
Marked as helpful
@A-amon
Posted
@markup-mitchell Thank you! Tho I had to manually adjust the viewbox values to get the right size π