Design comparison
SolutionDesign
Community feedback
- @rayaattaPosted 10 months ago
Hi ridhofaaauzi, nice solution 👏
I have some suggestions to help you improve your code.
1 Try to make your html more Semantic by wrapping the main page content inside a
<main>
tag . Replace<div class="container">
With<main class="container">
. This changes nothing visually but Using it makes all the difference. Using semantic markup improvesSEO
And user experience (accessibility) for people using assistive technology such as screen readers.
2 In order to get it looking closer to the design add
background-repeat:no-repeat; background-size:contain;
to the body css
I hope this helps.
Happy coding ✌️
Marked as helpful0
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