interactive-rating-component-main using flex

Community feedback

• 

  David•8,000

  @DavidMorgadePosted about 2 years ago

  Muy buenas Cristian, buen trabajo finalizando este challenge! enorabuena

  Lo único que veo que no funciona del todo correctamente es al hacer submit sin poner ningún tipo de rating, podrías añadir algún tipo de validación para que el usuario tenga que elegir un rating obligatoriamente, incluso mostrando un alert() si el usuario intenta hacer submit sin seleccionar un número.

  Modificando así tu código creo que lo podrías conseguir!

  btnSubmit.addEventListener("click", () => {
          if(rating.textContent === '') return alert('Choose a rating before submitting!')
  	thankyouContainer.classList.remove("hidden")
  	ratingContainer.style.display = "none"
  })
  

  Espero que mi feedback te sea de ayuda, sigue así que quitando lo de la comprobación, todo estaba perfecto!

  Marked as helpful

  1

  ---

  

  Cristian Eduardo Ledesma•90

  @CristianEduardoLedesmaPosted about 2 years ago

  hello @DavidMorgade, I did not work seem to overlook the condition, I could not find the error, also thanks for the help 👍👍

  1
• 

  Yacine Kahlerras•820

  @yacineKahlerrasPosted about 2 years ago

  Hello good job on the design i have one suggestion you can center your card by adding min-height: 100vh to the body and removing the margin-top: 50% from your 'container-rating'. happy coding !

  Marked as helpful

  0

  ---

  

  Cristian Eduardo Ledesma•90

  @CristianEduardoLedesmaPosted about 2 years ago

  hello @yacineKahlerras , thanks for the suggestion !! 👍

  1