Responsive website with css grid

Community feedback

• 

  Yazdun•1,310

  @YazdunPosted about 3 years ago

  Hi ! here I have some suggestions

  display:flex is a row by default so you don't need to declare it as a row

  I think this challenge can get done much easier by flexbox rather than grid

  try to learn BEM naming convention ( or any other conventions you prefer ). It will make your code easier to read and maintain.

  also I opened a pull request to your repo which will fix your background and scroll issue

  Marked as helpful

  1