Sunnyside agency landing page using GULP, SASS, BEM, HTML, GRID, FLEX

Community feedback

• 

  Aman Patil•885

  @Amanpatil-DevPosted over 3 years ago

  Hello Roman👋, Your solution looks good and responds well But some their are some things that i would like to point out:-

  1. The graphic design and photography content sticks to top of the div which has class plat--theme--cherry and plat--theme--orange respectively

  Solution for that would be .wrapper{ position:absolute bottom:0 } for both cherry wrapper and orange wrapper

  2. Lastly their are two divs like this <div class="plat"></div> <div class="plat"></div>

  I observed that when you to to screen width 1111px the images inside those divs with class plat__img adjust according to the min-height:37.5rem try adding height: 100% at width 1111px to class play__img and your will be good to go

  That's All👍 Hope this helps🙂

  Marked as helpful

  0

  ---

  

  Roman Zvir•290

  @rmzvrPosted over 3 years ago

  @Amanpatil-Dev thank you for your feedback, I really appreciate any constructive comments, as soon as I can fix it)

  1

  

  Aman Patil•885

  @Amanpatil-DevPosted over 3 years ago

  @rzvr most welcome 👍

  0