Made using HTML, CSS basics, Flexbox, Grid and mobile-first design
Design comparison
Solution retrospective
Hi!
Please Leave Some Feedback!
I would really appreciate it!
:)
Community feedback
- @tesla-ambassadorPosted about 3 years ago
This is an awesome solution! The responsiveness is really smooth! All I can say is that in order to get rid of the HTML issue, consider adding a header tag to your article or even better, you could just use a div and that way you don't need to add a header tag!
0@R0b3rtGPosted about 3 years ago@tesla-ambassador I've checked the HTML issue and I don't really know what it means because I have a heading in my header and in both articles. Anyway, thank you for the feedback! :)
0@tesla-ambassadorPosted about 3 years ago@R0b3rtG So, I checked out your code and I found out that the header tag you placed under the first article with class "art-1" was assigned to the section tag because the section tag also requires a header tag. Therefore they require another header tag that is a direct child node of that article tag (It doesn't necessarily have to be a direct child node but it shouldn't be under any section tag and any other article tag within that article tag) I really hope this is helpful!
Marked as helpful0@R0b3rtGPosted about 3 years ago@tesla-ambassador So if I change that section with a div it shoud be ok?
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