Design comparison
Solution retrospective
Did I have to add the symbols for the background? (I was unsure of this part but it was fun making it)
Community feedback
- @SinisaVukmirovicPosted almost 2 years ago
Hello!
Haha, no, you don't have to add the symbols, Those symbols are your in the "preview" images. You should make and try to looks as close to the "design" image. Looks good, though.
What you should do, is fix those report errors.
This "Document should have one main landmark" means you are missing <main> semantic element in your html. Landmark just means element.
This "All page content should be contained by landmarks" means all non-semantic elements should be inside of semantic elements, and not directly inside of <body> element.
Try to use semantic elements in your HTML. More about semantic elements
Hope this helps!
Marked as helpful1@pixelRenaPosted almost 2 years ago@SinisaVukmirovic Ahh okay haha funny I thought the background symbols had to be added! But thanks so much. I will look into those reports and check in with those semantic elements. Thanks so much for the advice and info!
0@SinisaVukmirovicPosted almost 2 years ago@pixelRena No problem.
Good job on fixing those errors!
Best of luck in the next projects.
0
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