3-column-preview-card-component-main responsive using HTML/CSS Flexbox

Community feedback

• 

  P

  Katrien Schuermans•1,210

  @katrien-sPosted about 3 years ago

  This looks good! It functions, the code is easy to read. There's not much to be added.

  • I would suggest however to look into developing mobile-first. No big deal on this excercise but it might be once you move on to bigger ones.
  • Make the transition from desktop to mobile view a bit more smooth. Use the flexibility of flexbox. Now it just jumps from one static size to another, by using max-width instead of width.
  • Add cursor: pointer; to the button.
  • Maybe use classes to the buttons: .btn, .btn-1, .btn-color, ... This way you can bundle code and avoid repetition.

  Keep going. This looks promising. Happy coding.

  Marked as helpful

  3

  ---

  

  Orlando Guevara•220

  @coding-tomatoPosted about 3 years ago

  @graficdoctor I'd also add to take the chance to learn about the BEM methodology to structure CSS classes! Not fully necessary given how small the project is, but I think BEM is a great addition to any CSS code.

  0

  

  Thainá Lopes•110

  @thainadlopesPosted about 3 years ago

  @graficdoctor Thank you so much for the tips! : )

  0