Skip to content
This repository was archived by the owner on Jul 11, 2026. It is now read-only.

DataLovers POKENDIA GO!#37

Open
YazTepRod wants to merge 20 commits into
Laboratoria:masterfrom
YazTepRod:master
Open

DataLovers POKENDIA GO!#37
YazTepRod wants to merge 20 commits into
Laboratoria:masterfrom
YazTepRod:master

Conversation

@YazTepRod

Copy link
Copy Markdown

se visualiza el ordenar y filtrar, pero no me funciona ninguno al dar click,
Quiza no estoy llamando bien a mis variables

También ya puede visualizarse más información sobre los pokémones, dandole click a la tarjeta, solo falta darle estilo

Comment thread src/data.js Outdated
if (abc.name > bcd.name) return 1;
if (abc.name === bcd.name) return 0;
return -1;
});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pregunta, a qué se refieren los parámetros abc, bcd? Me parece que podrían tener nombres más descriptivos :)

Comment thread src/data.js
sortData,
filterData
};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muy buen trabajo!

Comment thread src/index.html
@@ -20,21 +19,17 @@ <h1> INICIAR SESIÓN </h1>
<p><input id="Password" class="userData" type="text" name="Contraseña" placeholder="Contraseña"></p>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Es preferible escribir los id y clases en mayúsculas y elegir como hacerlo, si será en camel case todo en camel case o Pascal case, pero elegir solo uno para tener un mejor orden.

Comment thread src/index.html


<!-- Segunda pantalla Crear cuenta -->
<section id="sign-in" class="page2">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La clase del container debería ser más descriptiva y así evitamos añadir comentarios innecesarios.

Comment thread src/index.html Outdated
<p><button id="friend" class="click" type="button">Elegir acompañante</button></p>
<button id="test"></button>
<!-- <p><input id="Search" class="space" type="text" name="Buscar" placeholder="Búsqueda"></p>
<button id="test"></button> -->

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Código que no se utilice debemos borrarlo :)

Comment thread src/index.html Outdated
<option value="03"> A-Z</option>
<option value="04"> Z-A</option>
</select>
<select id="orderpokemon" class="type-select">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Elegir un idioma para nuestro código, de preferencia en inglés :)

Comment thread src/index.html Outdated
<span class="slider round"></span></p>
</div>
<div class="conteiner">
<h3>Zonas cercanas de Centros Pokemón</h3>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a typo, should be "container" :)

Comment thread src/main.js

//Devuelve el primer elemento que es #allThePokemones.
let list = document.querySelector ('#allThePokemones')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Los paréntesis deben estar junto al método:
...querySelector('');

Comment thread src/main.js Outdated

//Mostramos en consola los arrays
console.log (catalogue1);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Los console log solo nos sirven para debuggear, no hay que subirlos a github :)

Comment thread src/main.js Outdated
<p>${pokemon.type}</p>
<p>${pokemon.weaknesses}</p>
</div>
`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cuidar la indentación :)

@AdrianaHY AdrianaHY left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muy bien trabajo chicas!! cada vez van mucho mejor!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants