Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Equipo Anabel y Mónica #41

Open
wants to merge 45 commits into
base: master
Choose a base branch
from
Open

Conversation

ClaviHaze
Copy link

No description provided.

ClaviHaze and others added 30 commits February 28, 2020 13:07
… función automática. Se incluyeron las funciones nuevas en main.js y la función en data.js para el fetch y la creación de tabla
Added to Magnemite, Magnetron, Clefairy, Clefable, Jigglypuff, Wigglytuff and Mr. Mime
Add Fairy and Steel Types
…n icons.js de fight a fighting, creación tabla de detalles en index.html
…eadme, se cambio funció de highlight con un mouseover
unfiltered: () => {
const pokeObject = pokemons;
return pokeObject;
},
};
Copy link
Contributor

Choose a reason for hiding this comment

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

Excelente! Que código tan ordenado, muy bueno el approach de hacer los filtros en un objeto.

'2 km',
'5 km',
'10 km',
],
Copy link
Contributor

Choose a reason for hiding this comment

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

Felicidades por esto! es muy entendible.

<g id="Grupo_304" data-name="Grupo 304">
<g id="mega-bola" transform="translate(0)">
<g id="Grupo_303" data-name="Grupo 303" transform="translate(0)">
<g id="Grupo_288" data-name="Grupo 288" transform="translate(1.057 1.057)">
Copy link
Contributor

Choose a reason for hiding this comment

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

Recuerdad que puedes optimizar todas las imágenes, de ese modo no pesarán tanto. En internet puedes encontrar optimizadores. https://jakearchibald.github.io/svgomg/

src/index.html Outdated
<title>Data Lovers</title>
<title>PokeGo Space</title>
<link rel = "icon" type = "image/png" href = "https://raw.githubusercontent.com/ClaviHaze/CDMX009-Data-Lovers/master/src/images/pokemon.png">
<link rel = "apple-touch-icon" type = "image/png" href = "https://raw.githubusercontent.com/ClaviHaze/CDMX009-Data-Lovers/master/src/images/pokemon.png"/>
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<meta http-equiv="X-UA-Compatible" content="ie=edge">
Copy link
Contributor

Choose a reason for hiding this comment

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

Recuerdas ser consistente con los espacios.

const allPoke = document.querySelector('#allPoke');
const eggKm = document.querySelector('#eggKm');
const home = document.querySelector('#home');

Copy link
Contributor

Choose a reason for hiding this comment

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

Quizá se pueda optimizar si en lugar de un id les agregas una clase.

/*.pokeImg:hover{
transform: scale(1.5)
}*/

Copy link
Contributor

Choose a reason for hiding this comment

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

Recuerdas eliminar código que no se utiliza.

width: 30px;
margin-bottom: 12%;
}
.pokeType{
Copy link
Contributor

Choose a reason for hiding this comment

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

Deja un enter entre cada clase.


describe('example', () => {
// escribe aquí tu test
expect(typeof filters.filterByType).toBe('function');
});
Copy link
Contributor

Choose a reason for hiding this comment

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

Muy buejn acercamiento, por favor sigue practicando

Copy link
Contributor

@AdrianaHY AdrianaHY left a comment

Choose a reason for hiding this comment

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

Excelente trabajo! Tu código es muy ordenado y así da mucho gusto leerlo! Te recomiendo que sigas así, solo recuerda optimizar las imágenes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants