Responsive Loopstudios Landing page with HTML,CSS and javaScript

Community feedback

• 

  David•8,000

  @DavidMorgadePosted about 2 years ago

  Hello Olaitan, great job finishing the challenge! you did a great work in your solution

  If you don't mind, I would like to give you some feedback:

  • There is at some range of the page that is getting broke, from like 600px to 1000px, your div with the class content-1 overlays a bit the page, what you can do as a fast fix is to increase the media querie pixels so it changes the styles later, you should try to maybe adapt a bit the styles but setting the media a bit higher on pixels will fix the issue for now.

  • Apart from that your CSS is very good! I like the on hover effects on the images!.

  • The html is totally build out with divs! try using more semantically tags like header, main (divided in sections and articles), and also a footer, the html is a very important part of your webpage, it will determine the accesibility and the position on search engines, if you don't have a semantic html the search engines will penalize you and get your page as the latest results on search, go for those html tags man! they are easy to learn and you will get use with them in no time.

  Hope my feedback helped you, if you have any questions, don't hesitate to ask!

  Marked as helpful

  1

  ---

  

  Olaitan•230

  @Olaitan01Posted about 2 years ago

  @DavidMorgade OMG! Thank you so much. I did not know all of these. I will do the quick fix with your tips. This is more than helpful.

  1