Sunnyside challenge

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Awesome work with this project. Looking at your solution, I would like to suggest the following for you…

  • Clean the whitespaces in the code
  • Remove unnecessary code to keep it clean
  • Simplify the padding: 1rem 2rem 1rem 2rem; in #nav-contact into padding: 1rem 2rem;
  • Remove padding-top: 12%;, padding-left: 8%; and padding-right: 10%; in the .audience rule set and simplify it into padding: 12% 8% 10% 0;
  • In the media query max-width: 1420px, remove margin: 0;, margin-left: auto; and margin-right: auto; into margin: 0 auto;. Apply the same in method max-width: 992px breakpoint

  Above all, the design is well implemented. Keep it up and Hope this is helpful!

  Marked as helpful

  1
• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hi Attila, great job and congratulations on completing the challenge.

  One advice for you, try to add a little transition on the element with hover effect.

  Overall you did well :)

  Hope this help and happy coding!

  Marked as helpful

  1
• 

  Hexer•3,660

  @EmmanuelHexerPosted almost 3 years ago

  Great work overall man. Keep it up

  1
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  hay! Attila Nice work

  0