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

Entrega-Modulo-3-Tarea-01 #7

Open
Cjulianmd opened this issue Jun 10, 2022 · 5 comments
Open

Entrega-Modulo-3-Tarea-01 #7

Cjulianmd opened this issue Jun 10, 2022 · 5 comments
Labels

Comments

@Cjulianmd
Copy link
Collaborator

image
image
image

URL de despliegue: https://62a29a58134fd63db77a340d--ephemeral-clafoutis-ea8c0c.netlify.app//

fue un reto muy largo mas que todo,
use java scrip para poder pasar con las flchas las imagenes de las flechas las saque de una pagina de iconos lo cual pienso que hace mas comodo ala vista tube problemas para añadir la fuente de tecto pero nada que no se pueda solucionar separe y enumere cada una de las imagenes en columnas con un data-id en numeros para que java pudriera añadir un numero e ir contando asi hasat llegar al limite y ahi no poder seguir pasando para la informacion añadi una ventana para no tener que añadir otra pagina y asi fuera mas dinamico

@Gatc-18 Gatc-18 changed the title Entrega-Modulo-3-Tarea-01-Divisas Entrega-Modulo-3-Tarea-01 Jun 13, 2022
@Gatc-18
Copy link

Gatc-18 commented Jun 13, 2022

Hola @Cjulianmd , espero te encuentres muy bien, muchas gracias por tu entrega, sin embargo, tu reto no cumple con lo esperado y para poder ser aprobado es necesario que hagas/sigas las siguientes correcciones/recomendaciones :

  1. La información de los personajes NO puede estar dentro del HTML de forma directa, debes hacer uso de JS para el renderizado de la misma.
  2. Debes hacer uso de Módulos.
  3. Aplicar alguna desestructuración.
  4. Tener un arreglo con mínimo 10 objetos.
  5. Hacer uso del favicon.
  6. Hacer uso de la propiedad Object-fit.
  7. Apegarte más al diseño proporcionado.

Por favor cuando hagas lo sugerido, comenta nuevamente el issues y me etiquetas para proseguir con la revisión

@Cjulianmd
Copy link
Collaborator Author

Gatc-18

bueno sobre lo del html use un modulo para que la informacion pudiera estar en seccion.js en un formato tipo json al cual se le extrae la informacion que se valla nesesitando con respecto al codigo de jjava scrip no tube que realizar muchos cambios ya que lo unico que realize fue que la infomacion que estaba en el html estubiera en el java como sugeriste

@JoseGarcia03
Copy link

Hola @Cjulianmd , espero te encuentres muy bien, muchas gracias por tu entrega, sin embargo, tu reto no cumple con lo esperado y para poder ser aprobado es necesario que hagas/sigas las siguientes correcciones/recomendaciones:

  1. Hacer uso de la propiedad Object-fit.
  2. Hacer uso del favicon.

Por favor, cuando hagas lo sugerido, comenta nuevamente el issues y me etiquetas para proseguir con la revisión.

@Cjulianmd
Copy link
Collaborator Author

Cjulianmd commented Jun 17, 2022

JoseGarcia03

hola profe ya use el favicon y cambie el icono y el odject-fit lo use para cambiar la imagen de los personajes

image
image
image

@JoseGarcia03
Copy link

JoseGarcia03 commented Jun 21, 2022

Hola @Cjulianmd espero te encuentres muy bien, revisando tu reto, veo que cumples con todos los requerimientos funcionales planteados por la actividad, por ende, proceso a cerrar y aprobar este issue, te motivo a que día a día sigas aprendiendo y caminando hacia la excelencia.
PD: Recuerda que cuando realices un cambio debes también actualizar la URL de despliegue para poder ver esos cambios

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

No branches or pull requests

3 participants