Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @msa-sitare

    Posted

    Pretty nice, I liked ease in transition

    0
  • @msa-sitare

    Posted

    I'd recommend changing the image width from 60% to 100%, this way it lines up with the content nicely. Keep up the good work.

    Marked as helpful

    0
  • @msa-sitare

    Posted

    Your layout is structurally sound, but it lacks attention to detail in terms of proportions and spacing. Consider refining the dimensions, padding, margin, and alignment to ensure consistency. Good work, and with a little fine-tuning, it'll be even better!

    0
  • P

    @gabrieltrtl

    Submitted

    What are you most proud of, and what would you do differently next time?

    For this and other challenges, I confess that I would have liked to have used SASS, even without needing it.

    What challenges did you encounter, and how did you overcome them?

    I encountered difficulties with sizing this image, but I reached out to the FEM community for help and got the answers I was looking for.

    What specific areas of your project would you like help with?

    I would like to receive a code review, to know if I'm on the right track, I don't feel confident in my code yet

    @msa-sitare

    Posted

    Hey it's pretty good, you have very good attention to detail.

    my only recommendation would be to change drop-shadow to make it like the exact design. Something like this:

    @media (min-width: 650px) {
    
      main {
        max-width: 24rem;
        box-shadow: 8px 8px;
      }
    
      main:hover {
        box-shadow: 16px 16px;
      }
    }
    

    Right now it looks like the default state has 16px drop-shadow. Except for this nitpick, it's great.

    0