Design comparison
Community feedback
- @LesSylPosted 18 days ago
Hi! Your solution looks grate. I have a few suggestions:
-
the <a href ="">attribute should specify the destination of the link, you can read more here: w3- <a href>, MDN- <a href>,
-
the <img> tag should have the <alt> attribute completed. It is important for SEO. HTML: A good basis for accessibility,
-
it is good practice to use rem and em to specify font size and margins and padding. More information here: CSS values and units,
-
it is good practice to use a naming convention such as BEM (Block, Element, Modifier) is beneficial as it makes the CSS more organised, readable and easier to maintain. BEM helps to clearly understand the purpose of each class, avoid naming conflicts and create reusable components, leading to a more scalable code base. More details here: BEM.
Good luck!
0 -
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord