Design comparison
Solution retrospective
Thanks for checking out my solution. Would be grateful for any feedback and advice
Community feedback
- Account deleted
Hi there π
Congratulate on finishing your project π. You did a great job π‘
I give some suggestions to help you take your project design to the next level ππ
- Change the
h2
element font size to 2.6rem because right now it looks small
Happy coding β
Maqsud
Marked as helpful1@Iman-mohammadPosted over 2 years ago@maqsudtolipov Hi π Thank's for your advice, I modified it , it got better ππ
1 - Change the
- Account deleted
Hello there! π
Congratulations on finishing your challenge! π
I have some feedback on this solution:
-
Always Use Semantic HTML instead of
div
like<main>
<header>
, etc for more info -
Consider using the h1 element as a top-level heading only (all h1 elements are treated as top-level headings by many screen readers and other tools).
if my solution has helped you do not forget to mark this as helpful!
Marked as helpful1@Iman-mohammadPosted over 2 years ago@Old1337 Thank you for these tips, I will take them into consideration from now on βΊοΈ
1 -
- @BaymaGafurovPosted over 2 years ago
Great Job!
1
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