Responsive card component using CSS and Flexbox

Community feedback

• 

  hitmorecode•6,680

  @hitmorecodePosted about 1 year ago

  Congratulations well done. I took a look at your page and I have a few tips.

  • If you change the breakpoint to 650px you wont have so much empty space on the sides of the screen.
  • Right now you have five media queries. You don't need this much, you could have done it with just one media query

  1

  ---

  

  Stu Cowley•190

  @frontendstuPosted about 1 year ago

  @hitmorecode 👋

  Thanks for your feedback. I'll have a look into the 650px breakpoint. I'm not sure why I didn't use a smaller one in the first place. I'll adjust this.

  I write so many media queries to keep them organised with their respective classes. I'm not sure if this is bad practice so any advice on that is welcome ☺️

  0

  

  hitmorecode•6,680

  @hitmorecodePosted about 1 year ago

  @frontendstu The thing is all five media queries have the same min-width, so there is no point in using that many. The only time you use more media queries is when you want each media query to behave in a different way.

  1