Pakimanes#32
Conversation
| //export default { | ||
| "pokemon": [{ | ||
| "id": 1, | ||
| "num": "001", |
There was a problem hiding this comment.
Por qué comentaron el export?
There was a problem hiding this comment.
por que en el "pokemon.js" que ustedes proporcionaron no vienen separadas las categorías, steel y fairy, le preguntamos en el discord a Bliss y nos dijo que podíamos hacer otro paquete de datos y en base a ello se creo "pokemon1.js"
| </div> | ||
|
|
||
|
|
||
| <!--imagenes deslizables-------------------------------------> |
There was a problem hiding this comment.
Recuerden eliminar los espacios y comentarios innecesarios.
There was a problem hiding this comment.
eso fue culpa mia e igual de las otras lineas, lo que sucede es que mi compañera y yo teniamos problemas al estar armando paralelamente el proyecto entonces para evitar confusiones agregue todo eso pero puedo borrarlo.
| </div></div> | ||
|
|
||
| <!--Menu--------------------------------------------------> | ||
| <main> |
|
|
||
| <!--Search-------------------------------------> | ||
| <div class="box"> | ||
| <form> |
| //---------------------Clasificación por tipo de pokemon------------------------ | ||
|
|
||
| let btn1 = document.querySelector('#poison'); | ||
| btn1.addEventListener('click', mostrar1 ) |
There was a problem hiding this comment.
Los nombres de variables deben ser más descriptivos, qué hace este botón? Eso no puede dar una idea.
There was a problem hiding this comment.
este botón al dar click nos manda la información de los pokemones veneno pero podemos cambiarlo igual
| function mostrar17() { | ||
| document.querySelector('#root').innerHTML= ''; | ||
| typeSteel() | ||
| } |
There was a problem hiding this comment.
Quizá esto se pueda refactorizar para que ese evento se pueda hscer en cada botón pero con una función.
| } | ||
|
|
||
| let c = document.getElementById('C'); | ||
| c.addEventListener('click', all3) |
There was a problem hiding this comment.
Recuerden no nombrar variables solo con letras, no me dice nada.
AdrianaHY
left a comment
AdrianaHY
left a comment
There was a problem hiding this comment.
Muy buen trabajo! Por favor solo recuerden nombrar de mejor manera sus variables ya que el ponerlos como nombre una letra o algo tan genérico no me dicen nada, no me indica a que se refiere.
3 participants
Buenas tardes, solicito el pull request para nuestro proyecto "Bienvenido a Pakimanes", aun no terminamos el proyecto pero seguimos trabajando en el, nosotras hicimos otro set de base de datos porque la base de datos no estaba incluyendo los Pokémones por tipo "Fairy(Hada) y Steel(Acero)" y en esa base de datos estamos trabajando tiene por titulo "pokemon1.js".