Responsive - CSS - SASS

Community feedback

• 

  banana joe•135

  @bananaJoeRealPosted about 4 years ago

  Hey Andrea Lecuona!

  Your solution looks nice but it's not responsive.

  • Change the break point 375px to 620px it will be great! (For the header).
  • Add hover to the header links to be interactive with the user.

  Happy codding!

  1

  ---

  

  Andrea•270

  @AndreaLecuonaPosted about 4 years ago

  @bananaJoeReal Thanks for you feedback! I'll be fixing it very soon. Have a great day!

  1
• 

  Jadson Santos•235

  @santos-jadsonPosted about 4 years ago

  Hello Andrea, how you doing?

  Your code is really good and very similar to the original design. I don't have anything bad to say, I mean, the only thing I think you could change is the responsiveness, because when I open the site in a resolution bigger than 375px, the layout is breaking. But don't worry, your code is great and that's not a huge issue :D.

  1

  ---

  

  Andrea•270

  @AndreaLecuonaPosted about 4 years ago

  @santos-jadson Thanks for you feedback! I was so worried about getting the 1440 and 375 resolutions right, that I overlooked the intermediate sizes. Have a great day!

  0