Responsive design, SCSS, Webpack, HTML, CSS

Community feedback

• 

  Mikhail Nepomnyashchiy•260

  @michey85Posted over 2 years ago

  Hey! Nice job, man!

  I have couple of comments:

  • you need a default active status for links
  • delete .eraser-img tag, you don't need it at all - just add overflow: hidden to you sections and you will get the same result, but without extra div's
  • try to name your css classes more explicit than 'block1,2,3'
  • it is always better to use something unique - margins OR padding, for example the last paragraph (last week ...) now has both of it, but it's better to use just padding

  Try to add js logic for this. It will be fun!

  Good luck!

  0

  ---

  

  Denilson•220

  @DenilDenilsonPosted over 2 years ago

  @michey85 Thanks, Overflow:hidden is gold

  0