Data Lovers - pokémon #39
Conversation
| export const example = () => { | ||
| return 'example'; | ||
| }; | ||
|
|
There was a problem hiding this comment.
Excelente trabajo! Un comentario muy picky, hay que borrar los comentarios que no se utilicen.
| </div> | ||
|
|
||
|
|
There was a problem hiding this comment.
Recuerden borrar los espacios innecesarios :)
|
|
||
| <header> | ||
| <div class="logo"> |
There was a problem hiding this comment.
Recuerden que las clases deben ser más descriptivas, se deben nombrar en relación a lo que contienen y no a lo que hacen, container-2 no es muy descriptiva
|
|
||
| <div class="container container-2" id="pokemon-page"> | ||
| <!--Segunda parte menu--> | ||
|
|
||
| <div class="container-bar" id="btn-menu"> |
There was a problem hiding this comment.
La clase debe ser lo suficientemene descriptiva para que no se necesiten comentarios.
| <li><a>Hoenn</a></li> | ||
| </div> | ||
| <li><a>Hoenn</a></li> | ||
| </div>--> | ||
| </ul> | ||
| </li> | ||
|
|
There was a problem hiding this comment.
Si este código no se v a utilizar, debemos eliminarlo.
src/index.html
Outdated
|
|
||
|
|
||
| </header> | ||
|
|
There was a problem hiding this comment.
Hay que eliminar los espacios innecesarios :)
| <script src="data.js"></script> | ||
| <script src="main.js"></script> | ||
| <!-- <script src="data.js"></script>--> | ||
| <!-- <script src="main.js"></script>--> | ||
|
|
||
|
|
||
|
|
There was a problem hiding this comment.
Borrar código que no utilizamos! :)
| @@ -38,6 +45,7 @@ body { | |||
| background-color: rgba(255, 4, 4, 0.596); | |||
| padding: 30px; | |||
| border-radius: 30px; | |||
| font-family: "menlo"; | |||
| } | |||
|
|
|||
There was a problem hiding this comment.
Es preferible utilizar clases para dar estilos.
-Agregar funciones para la barra de navegación.
-Funciones para el buscador.
-Añadir responsive design.