sunnyside-agency-landing-page-main using HTML, CSS and JQuery

Community feedback

• 

  Lukasz•160

  @lukaasz555Posted almost 2 years ago

  Good job Himanshu, I like that you used semantic HTML (header, nav, footer). Clickable header is also good feature, but there is a still some features that you can improve, e.g.

  • you haven't done a view for mobile (good practice is to work mobile-first), I mean that your hamburger is not showing when I am resizing window;

  • LEARN MORE shoud be a clickable element (redirection isn't important at the moment) so you should use <a> or <button> instead of <h2>;

  • when I am resizing divs .five and .six I am loosing text;

  • ul.about & ul.media: it's similar to learn more. You should use <a> or <button> instead of <li>, of course, you can wrap e.g. <a> inside <li>;

  • the color of footer is wrong:P

  Keep coding:-)

  Marked as helpful

  0

  ---

  

  himanshukrmr•50

  @himanshukrmrPosted almost 2 years ago

  @lukaasz555 Thanks! for the feedback, will work on the points you have suggested. And for mobile view I have done only for 375px right now.

  1