Design comparison
Solution retrospective
Any kind of feedback is highly appreciated! I am eager to learn. I don't know why it's like that on the screenshot, when I view the site everything seems to be fine. Let me know what I did wrong thank you!
Community feedback
- @pikapikamartPosted about 3 years ago
I forgot, you can take a new screenshot of your solution. Maybe that will help your case!!
Marked as helpful0 - @pikapikamartPosted about 3 years ago
Hey, good work on this one. The layout both desktop and mobile is good. Though at point 1120px going down to the mobile layout breakpoint, the screen are now hiding the elements, thus creating a scrollbar.
Some suggestions would be:
- it would be great to always include a
main
element on a website. This will wrap all the element that would be part of your site, but make sure you are not nesting theheader
if you have a heading like navbars, andfooter
element. The structure would be:
header main footer
- the text
Reliable, efficient delivery Powered by Technology
could be wrap in a singleh1
element. Then you could wrap the second text in adiv
orspan
and style it so that it will be below the first text. - the text after the text mentioned above is most likely to be
p
tag rather thanh3
. - the heading text on the 4 boxes could have been only using
h2
tag. Also when creating website, even if you didn't do it in here, thankfully, remember to not jump from a heading tag, with 2 levels. Like starting fromh1
then the next heading tag ish3
orh4
. Increment only by one. - the text below the heading of the boxes should be
p
tags and not heading. Remember to only use heading tags, where you think a text is a topic that gives overview to a content or a text. - the
alt
text on the images could be left empty since they are just purely decoration. - lastly, just for
grid
additions, since you are usingdisplay: grid
maybe you can create like for example a 2x2 layout before transitioning to a one straight boxes. That would be awesome.
But still, good job on this one. But remember those about heading tags.
Marked as helpful0@ccbdpPosted about 3 years ago@pikamart Hey! Thank you so much for taking your time to comment and give suggestions on how I can improve! I would definitely take into account everything you stated to improve projects moving forward! I appreciate it!
0 - it would be great to always include a
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