Skip to content
Submitted 4 months ago

Recipe List Using Flexbox

Justin Fulkerson•80
@JF451
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Recipe page
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Community feedback

  • Dylan•290
    @dquinn089Posted 4 months ago

    Fantastic work! Your solution it practically identical, and your code it very well structured and written! In all reality there really aren't any issues with your solution. The only thing I would recommend is making use of the different elements that HTML has to offer, to create "landmarks" in your code, making it much more readable to other devs. For example: Instead of:

     <body>
    <div class="container">

    do:

    <body>
  <main>
    <div>

    You also have not included responsive design in your CSS, so your entire "container" element is being cut off on the left side of the screen. You can make any necessary adjustments to any of your element's properties by using CSS media queries which can be implemented at the end of your css like this:

    @media () {
    body {
        margin: 
    }
    main {
        padding: 
        border-radius: 
    }

    Hope this helps, happy coding!

    0
  • Mateus•60
    @mateusxczwPosted 4 months ago

    add more paddings to space the content! Nice job.

    0

Please log in to post a comment

Log in with GitHub

Join 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