Skip to content
This repository was archived by the owner on Jun 10, 2021. It is now read-only.

Añadido característica de cambiar páginas mediante las teclas y busqueda instantánea #4

Conversation

mrignacio1231
Copy link

Ahora puedes cambiar de página con las flechas direccionales izquierda y derecha.
Y ahora mediante un evento que detecta cualquier cambio en el formulario de búsuqeda, realizará una búsqueda.

Copy link
Owner

@coderdiaz coderdiaz left a comment

Choose a reason for hiding this comment

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

Creo que voy a poner un linter para el indentado jeje parece que tu manejas 4 espacios por tab y yo manejo 2 espacios por tab.

Podrías adaptar el indentado a 2 espacios y agregar el commit?

@mrignacio1231
Copy link
Author

Creo que ya logre cambiar toda la identación a 2 espacios

Copy link
Owner

@coderdiaz coderdiaz left a comment

Choose a reason for hiding this comment

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

Ok, dejame revisarlo

@@ -12,7 +12,7 @@
</div><!--/limits-->
<!--search-input-->
<div class="form-group pull-right">
<input class="form-control" type="text"
<input @keyup="searching" class="form-control" type="text"
Copy link
Owner

Choose a reason for hiding this comment

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

Esta caracteristica aún no la iba a colocar debido a que trabaja del lado del servidor, lo que implica que por cada acción realizada con el teclado lanzaria un refresh de la información con una nueva petición, aquí hay que ver una forma que se ejecute el evento cuando el usuario deje de escribir en el formulario de busqueda.

@coderdiaz
Copy link
Owner

coderdiaz commented Feb 1, 2017

Vale, vamos a hacer esto, voy a cerrar este Pull Request, ya que presenta muchisimos cambios, te voy a pedir nadamas que me envies el PR donde solo integres la caracteristica para hacer el cambio de pagina utilizando las teclas, como te mencione, por el momento no voy a integrar la funcionalidad de lanzar el evento cada que cambie el contenido del input.

Saludos.

@coderdiaz coderdiaz closed this Feb 1, 2017
@mrignacio1231 mrignacio1231 deleted the changepagewithkeyboard-feature branch February 2, 2017 02:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants