Responsive site using Sass and flex box

Community feedback

• 

  Hafizan Adli•1,190

  @hafizanadliPosted about 3 years ago

  You did great, almost similar to design, and responsive.

  What I found in your code is sometimes you use too much specificity selector. You shouldn't over qualifying selectors. for example don't do this

  .cards .column1 .card1 {
    your style
  }
  
  

  do this

  .card1 {
    your style
  }
  
  

  Marked as helpful

  2
• 

  Shahin NJ•1,190

  @SJ-NosratPosted about 3 years ago

  Hi Costi, Great looking solution!

  I'd use a little bit of semantic HTML and make <div class="card1 card"> into an <article>; since each card is a unique stand-alone content.

  Best of luck with your coding journey!

  Marked as helpful

  1