CSS flexbox

Community feedback

• 

  5nufkin•60

  @5nufkin

  Posted 7 months ago

  Hi there, Good job on this one, your website seems responsive and looks good! One thing I noticed right away was the body background color that's not exactly what you were asked to do, consider fixing this. About your html file - for better readability and structure, consider adding a header and a footer - in the header you could include the image and name of the recipe, then in main the recipe itself and in the footer you could put your note - "challenge by frontend mentor..." Also, consider naming classes in a more meaningful name that would help understand your code, c1, c2 and so on doesn't really deliver the message on what exactly this class is about. In addidtion, consider using sections and ids as well, ids are used for elements that you only have one of in your code, for example you could replace the c2 div with a section tag, it would look like this: <section id="preparation-time">....</section/ That would make your code much more readable. Your CSS looks good and it looks like you took a mobile-first approach which is a great way to design these days, very good! I hope this was helpful, keep going - you're on the right track!

  01