Good code
Marked as helpful
@MikDra1
Posted
Well done, here are some things to review 😊:
Overusing div tags: Try using more semantic HTML elements like <section>
, <header>
, and <article>
. It’ll help with both accessibility and SEO.
Neglecting responsive design: Make sure you're using media queries and following a mobile-first approach so your site looks great on all devices.
Inconsistent class naming: It’s easy to end up with a mess of class names. Consider using a system like BEM for better organization and scalability.
Using px for everything: Instead of px
for fonts and layouts, try using relative units like rem
or em
to make your design more adaptable to different screen sizes.
Forgetting alt text on images: Don’t skip the alt
attribute. It’s essential for accessibility and helps search engines understand what your images are.
Using fixed heights for elements: Setting fixed heights can cause overflow issues. Use min-height
or allow content to expand naturally to avoid problems.
Not using responsive images: Be sure to use srcset
or the <picture>
element to optimize images for different devices. This improves performance, especially on mobile.
Hope you found this comment helpful 💗💗💗
Good job and keep going 😁😊😉
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