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

Solution by @galeedgutierrez #53

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

Conversation

GaleedGutierrez
Copy link

I enjoyed this exercise. 😀

GaleedGutierrez and others added 20 commits December 26, 2023 04:18
Se modificó eslint y editorconfig, se agregaron los archivos de prettier y stylelint
Según si hay o no elementos en la todo list .main y .footer se ocultarán o mostrarán.
Ahora podemos agregar una nueva tarea
Tarea marcada como "complete" o "pending" según el estado del checkbox de la tarea
Al hacer doble click se puede editar la tarea selecionada

Se actualizó la versión de Playwright para poder usar la extensión de testing de VSCode.
…la edición.

Cuando el modo edición está activado, se deberían ocultar los otros elementos y se mostrará un input que contiene el título de la tarea pendiente, que debe estar enfocado.
Cuando se agrega o elimina una tarea, el contador cambia, aumenta o disminuye según sea el caso.
El botón de eliminar una tarea funciona.
Podemos limpiar todas las tareas ya completadas y ocultar el botón cuando no hay ninguna tarea completada.
Ahora podemos guardar, eliminar y editar las tareas.
Cuando reiniciamos el navegador, persiste la última información añadida y/o modificada.

Se modificó app.spec.js, ya que no contemplaba el estado de "pending" y había un error en donde el test fallaba por falta de un await en "Persistence" línea 247.
Se puede filtrar las tareas pendientes, completas y ver ambas a la vez.
@GaleedGutierrez
Copy link
Author

This is the deploy: https://galeedgutierrez.com/laboratorio-mydayapp-js/

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