-
Notifications
You must be signed in to change notification settings - Fork 226
Event delegation #353
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
Event delegation #353
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bien 👍 💯
me quedarón duras sobre Manejadores/Controladores [handlers
]
Prefiero controladores ya que "controlan" los eventos. 🤷♂️
Después ví que se puede usar Conmutar por el Toggle..
puse link de referencia para que lo veas.
Dejé mis comentarios en el código ;)
Saludos!
Please make the requested changes. After it, add a comment "/done". |
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Jeje, no voy a poner este coment en una linea perdida ¿cual de ellas?
controlador HANDLER es algo mas mundano, cositas que ponemos en una funcion pra "manejar" algo. lo vi en partes del repo y me parecio bien, pero esta mechado, hay de los dos. |
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+EzequielCaste@users.noreply.github.com>
/done |
De nada 🤣 |
El sync es importante para uds, "YOU ARE WELCOME" por revisar más de 400 archivos en una semana Este PR ya no me satisface, fue violentado en lugar de convencido. |
@EzequielCaste tenemos que aceptar la elección de @joaquinelio porque fue lo que dicidimos en el Glosario. Veo que el único dilema con este artículo es dicha palabra así que me parece innecesario hacer esperar más para aprobarlo. |
En los otros artículos está como "controlador".
Vamos a confundir al lector si cambiamos las palabras artículo en artículo.
…On Mon, Aug 31, 2020, 21:35 Valentina VP ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In 2-ui/2-events/03-event-delegation/2-sliding-tree/task.md
<#353 (comment)>
:
>
-- Only one event handler (use delegation)
-- A click outside the node title (on an empty space) should not do anything.
+- Solamente un manejador de eventos (usa delegación)
Lo mismo que mi comentario anterior. Parece que a elección de @joaquinelio
<https://github.com/joaquinelio> prefiere "manejador"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#353 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMLHWQS7GFZMWMREHA23KWDSDQ6U7ANCNFSM4QG36QZA>
.
|
@EzequielCaste Para ello quería unificar el glosario, pero como dijiste, hay palabras que quedan a elección del autor. NO PODEMOS FORZAR a los colaboradores. Quizás "handler" es una palabra que debe estar en la lista "elección del autor" en el glosario. |
Propongo que para estos casos usemos el Método Google:
Buscamos ambos términos y el que resulte con mayor número de resultados
será el término que usaremos.
"Controladores de eventos javascript" 700.000 resultados aproximadamente
"Manejadores de eventos javascript" 70.000 resultados.
Alguna otra idea?
…On Mon, Aug 31, 2020, 21:44 Valentina VP ***@***.***> wrote:
@EzequielCaste <https://github.com/EzequielCaste> Para ello quería
unificar el glosario, pero como dijiste, hay palabras que quedan a elección
del autor. NO PODEMOS FORZAR a los colaboradores. Quizás "handler" es una
palabra que debe estar en la lista "elección del autor" en el glosario.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#353 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMLHWQQ5CNVNNDEKEKEKGF3SDQ7YJANCNFSM4QG36QZA>
.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Arreglamos los "clic" que me hicieron doler los ojos y lo aprobamos. 😄
Please make the requested changes. After it, add a comment "/done". |
Por mi bien. No sé que opinan @maksumi y @joaquinelio . |
¿todavia aca? Por la palabra, estaba dipuesto a DISCUTIRLA, compulsivo... esta bien esta palabra? ok, eso, no es admisible ese autoritarismo |
metodo google no va, ya lo habia respondido hace mucho, no tiene la mas minima relacion, especialemte con diferentes categorias de palabras argumento absurdo: argumento jerarquia lo voy a meter en el side article de maksumi en vez del chiste malo de Ilya |
@vplentinax en las review ignoro "click" porque a nadie impresiona, edit: También es valido el procedimiento alternativo, simplemente ignorar el argumento y violentarme el texto sin permiso, ahorrarnos el incordio de tanto "suggest" |
Por ahi todavia no entendes lo que me molestó Podes insultarme, decirme inútil, ignorante, me hago el ofendido y te contesto, todo funny. Acá puse una posición, esperaba un argumento y en contestación dijiste que hice todo mal y me cambiaste el texto de prepo. Que te creas con la verdad puede ser gracioso, que actues como si fueras el jefe regañando a otros colaboradores por que no se hace a tu manera, Esto que pusiste tiene olor a argumento, pero llega un poco tarde. |
Y si está mal, no lo uso aunque google le de ok |
Handler es una pequeña funcion mundana, la que usamos para manejar nuestras cosas. controlador es algo grosso, una interfaz que manejja un dispositivo pero me estoy yendo de tema Una frase que encontre por ahi
que dicho sea de paso no son sinonimos. |
Hace un mes me quedaría intentando que avancemos aquí siguiendo la conversación, pero ya no tengo tiempo para ello. Ya hice mi revisión, ya ví que no se aceptó, pues... prosigan como crean conveniente. Edit: |
ya etá roto, no tenia gana de eguir |
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por mi bien
Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉 |
No description provided.