@zineb-Bou
Posted
Your solution looks good on different screen sizes.
-
Setting h1 as a card title may look normal on this task since the card is the only component on the page, though this card would be part of a page in real production and a website should contain only one h1, so avoid choosing h1 as a card title you could choose h2 instead.
-
The social media icons are actually links, so you should put <a/>, add text inside it and make it sr-only then hide the SVG icons from the screen reader since they are purely decorative
happy coding 🙂
@dusanlukic404
Posted
@zineb-Bou yeah, of course on big project I would use only one h1. Thanks for such a helpful feedback 😀