HTML, CSS & JAVASCRIPT

Community feedback

• 

  Ctrl+FJ•810

  @FlorianJourdePosted over 1 year ago

  Hey @Aliha7ish ! That's a good solution !

  Be careful, your .main-heading goes behind your .main-header when screen size is under 700 px ! Maybe some padding-top could be added ?

  As well, do your know you can set a link with anchor on your arrow down ? Look at something like that :

  <a class="arrow" href="#features-section"><img src="img/arrow.png"></a>
  

  You'll be able to scroll down to the next section with this ID, just by clicking on the hyperlink element ! Just add a html { scroll-behavior: smooth; }, and you've got a nice scrolling effect really easily ! Pretty cool, huh ?

  One last thing : I think it's a better practice to put properties like font-size or overflow-x on the body tag, instead of html.

  But it's still pretty good, your responsive is also clean !

  Keep up coding !

  Marked as helpful

  1