Grid and flexbox component

Community feedback

• 

  Ediongsenyene Joseph I.•1,675

  @IEdiongPosted over 3 years ago

  Nice one Ayoub,

  • Work on your

  @media (min-width: 375px)
  .container {
  }
  

  It kicks in too early.

  • Also, look into the accessibility issues raised in the report above.

  I hope this was helpful, Cheers 🥂

  Marked as helpful

  1

  ---

  

  Ayoub1Dev•195

  @Ayoub1devPosted over 3 years ago

  @IEdiong Can you explain a little bit more what do you mean by it kicks too early. ''' else { thanks 😊 } '''

  0

  

  Ediongsenyene Joseph I.•1,675

  @IEdiongPosted over 3 years ago

  @Ayoub1dev I meant that you should have your first break point at 375px. There isn't much breathing space for the card at 376px to about 900px.

  • Make your first media query above 1000px.

  0
• 

  Carlos Guzman•285

  @guztrilloPosted over 3 years ago

  Hey Ayoub, nice job. Check responsive design, doesn't work properly.

  1

  ---

  

  Ayoub1Dev•195

  @Ayoub1devPosted over 3 years ago

  @guztrillo Thanks for your time, I will.

  0