@A-amon
Posted
Hello! Great work, so is the responsiveness.
I think using a button for .share would be much better. It will let screen reader users know it's a button to be clicked on. You could also try using checkbox instead (You can use CSS combinator to toggle showing/hiding .social-icons instead of using JS) but I am not sure if it's suitable, especially when told to screen reader users.
Seems like you are using margin to vertically align each item in .social-icons. Flexbox has a CSS align-items property for it.
Marked as helpful
@sandyivan
Posted
@A-amon copy that, Thanks, Amon!