Design comparison
Community feedback
- @MikDra1Posted about 1 month ago
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.
-
Ignoring accessibility: Focus on features like keyboard navigation, contrast ratios, and ARIA labels. They make your site usable for more people.
-
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 πππ
0 -
- @ElenakathPosted about 2 months ago
Colors, fonts, and spacing look great. Try making the image bigger.
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