Design comparison
Solution retrospective
If you spot anything you feel should be fixed/improved, please let me know!
Community feedback
- P@danielmrz-devPosted about 1 year ago
Hello @aryianj!
Your solution looks great!
I have a suggestion for improvement:
- For semantic reasons, use
<main>to wrap the main content instead of a<div>.
📌 This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, great job!
0 - For semantic reasons, use
- @Abhishek-G06Posted about 1 year ago
That's an excellent work. Keep it up.
Still, I'd like to raise a few points to make sure it helps you:
In HTML, rather than using <br> tag to break the sentence, try using the CSS on the paragraph and create a section using padding and margin values.
That will make the code more clean.
I hope It helps!
0
Please log in to post a comment
Log in with GitHub
Join 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