Responsive landing page using html, css and a little of javascript

Community feedback

• 

  Ivan•2,610

  @isprutfromua

  Posted about 3 years ago

  Hi there. You did a good job 😎

  keep improving your programming skills🛠️

  your solution looks great, however, if you want to improve it, you can follow these steps:

  ✅ Use a single variable naming style. ** submit_boton ** ** containerDos **

  const submit_boton = document.querySelector ('. submit');
  const containerUno = document.querySelector ('.container1');
  const containerDos = document.querySelector ('.container2');
  const raitingBtns = document.querySelectorAll ('. number');
  const res_calificacion = document.querySelector ('.calificacion');
  let calificacion_estrellas = 3
  

  ✅ Pay attention to the formatting of your code, or use automatic formatting

  raitingBtns.forEach (btn => {
  btn.addEventListener ('click', handleRatingBtnClick);
  });
  

  ✅ Try not to place the debug code in the repository

  console.log (calificacion_estrellas);

  ✅ Try to avoid importing fonts to css. It is better to place them in the title of the page using the link tag.

  ✅ css is also poorly formatted

  
  .encabezado {
    background-color: var (- Very-blue);
    border-radius: 50%;
    width: 45px;
  height: 45px;
  display: flex;
  justify-content: center;
  align-items: center;
  margin-bottom: 12px;
  margin-top: 12px;
  }
  
  .respuesta {
  
  display: none;
  
  }
  
  

  I hope my feedback will be helpful. You can mark it as useful if so 👍

  Good luck and fun coding 🤝⌨️

  00

  Marked as helpful
• 

  Raul Olvera•250

  @rolvera22

  Posted about 3 years ago

  Thank you. This will help me to improve my coding skills. Thanks for passing by!

  00