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

Event delegation #353

Merged
merged 26 commits into from Sep 20, 2020
Merged

Conversation

joaquinelio
Copy link
Member

No description provided.

Copy link
Contributor

@EzequielCaste EzequielCaste left a 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!

2-ui/2-events/03-event-delegation/2-sliding-tree/task.md Outdated Show resolved Hide resolved
2-ui/2-events/03-event-delegation/3-sortable-table/task.md Outdated Show resolved Hide resolved
2-ui/2-events/03-event-delegation/3-sortable-table/task.md Outdated Show resolved Hide resolved
2-ui/2-events/03-event-delegation/article.md Show resolved Hide resolved
2-ui/2-events/03-event-delegation/article.md Outdated Show resolved Hide resolved
2-ui/2-events/03-event-delegation/article.md Outdated Show resolved Hide resolved
2-ui/2-events/03-event-delegation/article.md Outdated Show resolved Hide resolved
2-ui/2-events/03-event-delegation/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

joaquinelio and others added 9 commits August 27, 2020 21:05
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>
@joaquinelio
Copy link
Member Author

joaquinelio commented Aug 29, 2020

@EzequielCaste

Jeje, no voy a poner este coment en una linea perdida ¿cual de ellas?

Edit
"destacar", ok, me agradó. En realidad me acordaba de los resaltadores, me equivoque de palabra.
"conmutador" ES la correcta, no la recordaba pero es essa sin duda, asi lo pedía cuando era técnico.
"controlador"... como técnico, me cuesta usarlo, para mí es el "driver" de disspositivos.
"propagar"... mal ahí, ¡todo el mundo lo usa! Cómo dijiste que aceptabas burbujeo. Shame on you.

controlador
Es algo grosso, me desagrada usarlo. Para mí, tecnico y programador,
los DRIVERS son los controladores que se programan para los dispositivos, algo muy especializado

HANDLER es algo mas mundano, cositas que ponemos en una funcion pra "manejar" algo.
captar el evento y manejarlo me parece mass natural

lo vi en partes del repo y me parecio bien, pero esta mechado, hay de los dos.
/done I guess...

joaquinelio and others added 6 commits August 29, 2020 03:03
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>
@joaquinelio
Copy link
Member Author

/done

@joaquinelio
Copy link
Member Author

De nada

🤣

@joaquinelio
Copy link
Member Author

El sync es importante para uds, "YOU ARE WELCOME" por revisar más de 400 archivos en una semana
Tomense el trabajo de mirar los 75 que quedaron

Este PR ya no me satisface, fue violentado en lugar de convencido.

@vplentinax
Copy link
Contributor

vplentinax commented Sep 1, 2020

@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.

@EzequielCaste
Copy link
Contributor

EzequielCaste commented Sep 1, 2020 via email

@vplentinax
Copy link
Contributor

@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.

@EzequielCaste
Copy link
Contributor

EzequielCaste commented Sep 1, 2020 via email

Copy link
Contributor

@vplentinax vplentinax left a 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. 😄

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@vplentinax
Copy link
Contributor

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 .

Por mi bien. No sé que opinan @maksumi y @joaquinelio .

@joaquinelio
Copy link
Member Author

¿todavia aca?
no se entiende:

Por la palabra, estaba dipuesto a DISCUTIRLA,
lo que NO es admisible es que ante mi argumento
la respuesta fuera modificarme el texto compulsivamente

compulsivo... esta bien esta palabra?
drae: si

ok, eso, no es admisible ese autoritarismo
NUNCA impuse nada, fui flexible en todo, y esto podia cambiarse
no sos jefe tipo siglo XIX para decir qué poner ni dónde se puede discutir ni cerrar issues sin resolver porque a vos no te gustan. Me criticaste la falta de "por favor". Como jefe nunca pedi por favor, pero tampoco impuse algo violentamente.

@joaquinelio
Copy link
Member Author

metodo google no va, ya lo habia respondido hace mucho, no tiene la mas minima relacion, especialemte con diferentes categorias de palabras

argumento absurdo:
-En la cabeza tenes puesta una camissa
-no, es un sombrero
google....
-gano la camisa: tenes puesta una camisa

argumento jerarquia
-tenes puesta una camisa (donde se debe)
-no, tengo puesta ropa.
-pero es una camisa
google...
-ok, es ropa, no es camisa.

lo voy a meter en el side article de maksumi en vez del chiste malo de Ilya

@joaquinelio
Copy link
Member Author

joaquinelio commented Sep 1, 2020

@vplentinax
y la onomatopeya de ladrido es "wof!"

en las review ignoro "click" porque a nadie impresiona,
pero la palabra bien castellana es clic, en España y Latinmmerica
"sorry" si te duele

edit:
jjjaajj fundeu no me abandona
www.fundeu.es/consulta/hacer-clic-1753/

También es valido el procedimiento alternativo, simplemente ignorar el argumento y violentarme el texto sin permiso, ahorrarnos el incordio de tanto "suggest"

@EzequielCaste
Copy link
Contributor

20 mil resultados
image

90 mil resultados
image

el método google para mí SI VA. es la principal herramienta que usamos como programadores y Estudiantes.
Cuando no sabemos de un tema lo buscamos en Google. Prefiero que los estudiantes encuentren correctamente el tema en cuestion..

@joaquinelio
Copy link
Member Author

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,
o peor acá, pisando trabajo sin permiso, ya no.

Esto que pusiste tiene olor a argumento, pero llega un poco tarde.

@joaquinelio
Copy link
Member Author

Y si está mal, no lo uso aunque google le de ok
Mejor pongo "event handler" y que active el autotraductor, varios millones de resultados tendran

@joaquinelio
Copy link
Member Author

Handler es una pequeña funcion mundana, la que usamos para manejar nuestras cosas.
por eso lo llaman handler,

controlador es algo grosso, una interfaz que manejja un dispositivo
lo malo que el ingles no es controller sino driver
y controller encima en otros lenguajes es especificamente otra cosa

pero me estoy yendo de tema
el tema es, resultado por consenso, no sos el jefe. El maintainer es el portero, salvo que seas quien paga a quienes colaboran aca

Una frase que encontre por ahi

Creo que cada traductor puede utilizar libremente la palabra matriz, arreglo o vector. Son términos que hacen referencia a lo mismo y se usan indistintamente. Si bien en la comunidad se utiliza comúnmente array, no debería haber una regla que no nos permita usar una palabra que CLARAMENTE es un sinónimo y no cambia el sentido de lo que queremos expresar.

que dicho sea de paso no son sinonimos.
array es arreglo,
vector es un arreglo unidimensional, y
matriz no puede ser unidimensional, tiene 2 o mas dimensiones

@vplentinax
Copy link
Contributor

vplentinax commented Sep 1, 2020

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:
@joaquinelio obvié el "click", pero entre mis sugerencias había otro tipo de cambios. Revisalo para avanzar por el lado de mi review.

@joaquinelio
Copy link
Member Author

ya etá roto, no tenia gana de eguir

joaquinelio and others added 4 commits September 4, 2020 19:17
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>
Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

Por mi bien

@vplentinax vplentinax merged commit 0f685c6 into javascript-tutorial:master Sep 20, 2020
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

@joaquinelio joaquinelio deleted the deledeve branch September 20, 2020 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants