Responsive card using Grid and Flexbox

Community feedback

• 

  Tiago José•300

  @tiago-jv0Posted over 2 years ago

  Hello antonio, i saw your solution and i'd like to give you some tips:

  • Use custom radio inputs instead of buttons for your ratings section, that way you will no longer need to switch styles using javascript ( in case you don't know about custom radio inputs, i recommend this tutorial : https://www.youtube.com/watch?v=5K7JefKDa4s)

  • Improve your html semantics, you are using a lot of div elements, there are a lot of semantic tags that you could use in your solution, for example : main, section , button (for your submit button)

  • Try to give a chance to BEM css methodology for your css classes, it will improve your code readability a lot.

  • Try to use sass/scss, it will improve you developer experience when writing css

  0