Mobile-first approach, semantic HTML5, CSS3, BEM, Flex

Community feedback

• 

  Mikołaj Szymczuk•435

  @mikolajszymczuk1Posted over 3 years ago

  Good job ;) but if you want to break the styles into smaller parts I recommend SASS, LESS etc. because you have to remember that each additional .css file is an additional request of the browser

  Marked as helpful

  0

  ---

  

  Denis Pankov•25

  @idenispankovPosted over 3 years ago

  @mikolajszymczuk1 Thanks to taking a time to review my work, i thought that is so small, so 2 break points enough?))) never used sass or less

  0
• 

  dewslyse•3,025

  @dewslysePosted over 3 years ago

  Hello Denis👋! Awesome job with your implementation 🔥🔥. Aside the background image, everything is nearly spot-on 👌. Good job.

  Marked as helpful

  0

  ---

  

  Denis Pankov•25

  @idenispankovPosted over 3 years ago

  @dewslyse Thank you for you time, any suggestion on a background image?))

  1

  

  dewslyse•3,025

  @dewslysePosted over 3 years ago

  @idenispankov

  I think you may have used the mobile svg for the desktop. I replaced your current img with pattern-background-desktop.svg using dev tools and it was well positioned.

  1

  

  Denis Pankov•25

  @idenispankovPosted over 3 years ago

  @dewslyse Got it, fixed a bit, Thanks

  0