Design comparison
Community feedback
- @anoshaahmedPosted almost 3 years ago
Hey good job on this challenge! Your report above shows a few issues. You can fix them by
- wrap everything in your body in
<main>
... OR use semantic tags! .... you can also giverole=""
to the direct children of your<body>
but that's a little frowned upon... Read more here - have at least one
<h1>
in your code
:)
Marked as helpful1@GuillermoOMPosted almost 3 years ago@anoshaahmed Thank you!
I applied your fixes and the report indicates everything to be fine now.
1 - wrap everything in your body in
- @rsrclabPosted almost 3 years ago
Hi, @GuillermoOM ~
Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.
Here are some of the tips I like to provide.
- Please try BEM for naming element classes. It will help you a lot on bigger projects.
https://www.frontendmentor.io/solutions/my-first-solution-on-chanllenge-V-4IzAivH
Here is my solution to this challenge, and if it can help you even a bit, it would be happy to me.
Cheers ~
Marked as helpful1@GuillermoOMPosted almost 3 years ago@tymren608 Thank you, likewise with yours!
Also, thanks for your input on the BEM theory, will def try to implement it on my future challenges and projects.
Also, I found your HTML to be quite well structured, good job on that
0 - @TMJSMC53Posted almost 3 years ago
Good job, Guillermo
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