Skip to content

Event delegation - #353

Merged
vplentinax merged 26 commits into
javascript-tutorial:masterfrom
joaquinelio:deledeve
Sep 20, 2020
Merged

vplentinax merged 26 commits into
javascript-tutorial:masterfrom
joaquinelio:deledeve

Conversation

@joaquinelio

Copy link
Copy Markdown
Member

No description provided.

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor

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!

@javascript-translate-bot

Copy link
Copy Markdown

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

joaquinelio commented Aug 29, 2020 •

Copy link
Copy Markdown
Member Author

@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
Copy Markdown
Member Author

/done

@joaquinelio

Copy link
Copy Markdown
Member Author

De nada

🀣

@joaquinelio

Copy link
Copy Markdown
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

vplentinax commented Sep 1, 2020 •

Copy link
Copy Markdown
Contributor

@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

EzequielCaste commented Sep 1, 2020 via email

Copy link
Copy Markdown
Contributor

@vplentinax

Copy link
Copy Markdown
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

EzequielCaste commented Sep 1, 2020 via email

Copy link
Copy Markdown
Contributor

@vplentinax vplentinax left a comment

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown

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

@vplentinax

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

joaquinelio commented Sep 1, 2020 •

Copy link
Copy Markdown
Member Author

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

vplentinax commented Sep 1, 2020 •

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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>

@vplentinax vplentinax left a comment

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown

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.

4 participants