Interactive rating component with HTML, CSS and JS

Community feedback

• 

  Luka Glonti•3,440

  @lack21Posted over 1 year ago

  Good work 👍, but I have some recommendations!

  • Replace height: 100vh to min-height: 100vh in the body, the difference is that, when you set height: 100vh to something, that means it won't be bigger than that, it might cause some problems in the future, so better approach is to set min-height: 100vh, like this in case content is overflowing container will adjust to it!
  • Replace width: 415px in the .card with the width: min(415px, 90%), like this when screen width is smaller than .card, .card will adjust to it, and also remove width: 330px when it goes on smaller screens!

  Marked as helpful

  0

  ---

  

  develoba•270

  @develobaPosted over 1 year ago

  @lack21 you're right, I'll change it! Thanks for the help, I still need to learn best practices for responsiveness.

  0