Hi, This looks really good!
You could definitely do this with one share block but theres nothing wrong with how you've done it now.
The only things that need addressing are accessibility related really:
- alt text on the main image should be describing the image if the image is meaningful, or empty if it is not
- all the buttons need accessible names (aria-labels). All the social ones would probably be anchor tags (links) not buttons too.
That's all from me. I'm really looking forward to seeing what you submit next ☺
Ps. Remember to upvote comment if you find it helpful, thanks
@cyishere
Posted
@grace-snow Thank you, Grace. I'll fix these later.
@cyishere
Posted
@grace-snow Hi, Grace. I fixed the accessibility issues. But about the social buttons, I think when the user press it, it would call a function to open a new window to write sth first then share, so I keep them buttons. (Although I need to add those functions either.😄) Thank you again. 🙏🏻
@cyishere they would still have to be interactive html elements. Icons themselves don't get clicked, the interactive tag wrapping them is what a user focuses on / clicks / taps / uses keyboard or other control to press. ☺
