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

Fusión de varios componentes de vista principal #83

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

jcardenas1989
Copy link

@jcardenas1989 jcardenas1989 commented Mar 2, 2024

Cordial saludo,
Dejo el enlace del demo de proyecto : https://jcardenas1989.github.io/curso-frontend-developer-javascript-practico/

Adicional a los temas tratados del curso, se adiciono componentes como:
*reflejar el producto seleccionado en la lista de detalles de compra(my order) con los detalles de: imagen del producto, precio o valor del producto, detalle del producto.

  • si el usuario agrega a la lista detalles de compra(my order) x cantidad de productos y desea agregar otro producto, lo podra realizar sin problema alguno.
  • se agrega al detalle de compra el valor total por los productos agregados al carrito.
    los archivos empleados (html) empleados para la fusión fueron:
    Clase11.html
    Clase7.html
    Clase8.html
    Clase13.html
    Clase6.html
    Clase12.html
    Nota: quedaron faltando detalles para mejorar la aplicación, sin embargo aplico los conocimientos adquiridos en el curso para realizar una contribución al proyecto.

calse finalizada
se deja funcional los eventos de menu
se replica detalles en el evento de lista de producto
commit de seguridad
se adiciona el evento de click y muestra la cantidad de productos agregados al carrito
se adicionan los productos a la lista detalles del carrito.
se agrega evento de limpiar el contenedor del detalle de los productos para evitar que duplique la informacion de los prodcutos
finalizamos adicionando a la lista del carrito el valor total de los productos a comprar
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.

1 participant