sunnyside landing page

Community feedback

• 

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  Hi Adeline,

  Great work on this project! Looks great for both desktop and mobile views. Just some suggestions I would like to share :

  • You can change the width of your project container to 100% instead. This will remove the scroll bar on the x-axis.
  • Don't forget to put the hamburger icon back to its original state when you press it again. It still remains to be tilted even after pressing it several times.

  I hope it helps :)

  Marked as helpful

  1

  ---

  

  adeline•170

  @adelinejunePosted about 3 years ago

  @fidellim Hi, Thankyou very much for the suggestion !

  1
• 

  Carlos•1,110

  @Carlos-A-PPosted about 3 years ago

  Hello

  This looks pretty great and good job! I do have a wide monitor and I would suggest setting the "graphic design" and "photography " as a div with background images instead of inserting the image in your HTML. That way you don't have to use absolute positioning on your text and make the element responsive on bigger screens too.

  1

  ---

  

  adeline•170

  @adelinejunePosted about 3 years ago

  @Carlos-A-P Thankyou for your suggestion I have already change to display grid so now it can fill the whole screen :)

  0