interactive-rating-component-main

Community feedback

• 

  Elaine•11,400

  @elaineleungPosted about 2 years ago

  Hi Kaan, well done on your second challenge, and I think this will look more like the design screenshot if it's a bit more centered. To do that, just remove the height property on the body and add min-height: 100vh, and since you already have flexbox with the correct properties on the body, that should work!

  The component also works well; one thing I suggest is to add an if statement to handle what happens when the user clicks submit without selecting a score. Right now when I do that, I still get brought to the thank you page. Also, the thank you image seems to be missing, so might want to fix the path there. Lastly, I would recommend not using a fixed percentage width as they can be hard to control, or if you do, combine it in min(), like this:

  .card-container{
     width: min(90%, 25rem);
  }
  
  // at the media query
  
  .card-container{
     width: min(90%, 30rem);
  }
  
  

  Other than that, great work!

  Marked as helpful

  0