Branch mio #27
Branch mio #27
Conversation
tareas/clase-5/mi-tarea-clase-5.js
Outdated
| const edad = Number(document.querySelector('#edad').value); | ||
|
|
||
|
|
||
| document.querySelector('#datos-finales').value = $datosFinales; |
There was a problem hiding this comment.
document.querySelector('#datos-finales').value = $datosFinales; $datosFinales hace referencia al botón, Vos estás diciendo que el elemento que tiene id datos-finales va a ser igual al botón 🤷♂
There was a problem hiding this comment.
En realidad vos tenés que decir que el valor de #datos-finales va a ser la concatenación de todos los otros datos que ya agarraste arriba.
document.querySelector('#datos-finales').value = `Te llamás ${primerNombre} ${segundoNombre} ${apellido} y tenés ${edad} años`
There was a problem hiding this comment.
entonces debería ser igual a $entrar?
There was a problem hiding this comment.
$entrar es un botón. Vos no podés decir que el valor de un campo de texto es igual a un botón. Me explico? En el comentario de arriba te puse exactamente lo que tenés que poner. Decime si no se entiende.
|
EL PR lo hiciste al repositorio de r-argentina-programa, debería ser de tu branch a tu propio master. Mirate este video: https://drive.google.com/file/d/1u1DKwJm_yjzlxALkj8oExb590PTTCog2/view |
|
Por otro lado, los commit messages no pueden ser "hace cambios" tienen que describir qué cambiaste y por qué. |
| const $ver = document.querySelector('#ver') | ||
|
|
||
| $ver.onclick = function(){ | ||
| const promedio = |
| const segundos = document.querySelector('#segundos').value; | ||
|
|
||
| const tiempo = document.querySelector('strong'); | ||
| tiempo.textContent = `${horas} hora/s, ${minutos} minuto/s y ${segundos} segundo/s`; |
There was a problem hiding this comment.
Esta función debería calcular la sumatoria de las horas, minutos y segundos de varios videos, no de sólo 1.
No description provided.