GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Event delegation by joaquinelio · Pull Request #353 · javascript-tutorial/es.javascript.info

Conversation

@joaquinelio

@joaquinelio joaquinelio commented Aug 21, 2020

Copy link
Copy Markdown
Member

No description provided.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@joaquinelio
Event delegation
234b419
@javascript-translate-bot javascript-translate-bot added the review needed label Aug 21, 2020
@iliakan iliakan mentioned this pull request Aug 21, 2020
Spanish Translation Progress #17
Open
EzequielCaste
EzequielCaste requested changes Aug 27, 2020
View reviewed changes

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor

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!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread 2-ui/2-events/03-event-delegation/2-sliding-tree/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/3-sortable-table/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/3-sortable-table/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot removed the review needed label Aug 27, 2020
@javascript-translate-bot

javascript-translate-bot commented Aug 27, 2020

Copy link
Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot added the changes requested label Aug 27, 2020
joaquinelio and others added 9 commits August 27, 2020 21:05
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/2-sliding-tree/task.md
e06394e
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/3-sortable-table/task.md
5dc484e
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/3-sortable-table/task.md
ff39c36
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
475c3e7
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
5e45108
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
84c78f0
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
b23c55e
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
9517ef9
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/article.md
7e24604
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio

joaquinelio commented Aug 29, 2020 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

joaquinelio and others added 6 commits August 29, 2020 03:03
@joaquinelio
Update article.md
f1cf397
@joaquinelio
Update task.md
031c98a
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/article.md
c213810
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/article.md
4848363
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Update 2-ui/2-events/03-event-delegation/article.md
4c69af1
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio @EzequielCaste
Apply suggestions from code review
41c7660
Co-authored-by: Ezequiel Castellanos <[email protected]>
@joaquinelio

joaquinelio commented Aug 30, 2020

Copy link
Copy Markdown
Member Author

/done

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot added review needed and removed changes requested labels Aug 30, 2020
joaquinelio and others added 4 commits August 30, 2020 05:27
@joaquinelio
"propagación" de eventos
789814d
@EzequielCaste
Update 2-ui/2-events/03-event-delegation/article.md
2ec1979
@EzequielCaste
Update 2-ui/2-events/03-event-delegation/article.md
1f09fd1
@EzequielCaste
Update 2-ui/2-events/03-event-delegation/article.md
37fc3f3
@joaquinelio

joaquinelio commented Aug 31, 2020

Copy link
Copy Markdown
Member Author

De nada

🤣

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Aug 31, 2020

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vplentinax

vplentinax commented Sep 1, 2020 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@EzequielCaste

EzequielCaste commented Sep 1, 2020 via email

Copy link
Copy Markdown
Contributor
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> .
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vplentinax

vplentinax commented Sep 1, 2020

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@EzequielCaste

EzequielCaste commented Sep 1, 2020 via email

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> .
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

vplentinax
vplentinax requested changes Sep 1, 2020
View reviewed changes

@vplentinax vplentinax left a comment

Copy link
Copy Markdown
Contributor

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread 2-ui/2-events/03-event-delegation/2-sliding-tree/solution.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/2-sliding-tree/solution.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/2-sliding-tree/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/2-sliding-tree/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/3-sortable-table/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 2-ui/2-events/03-event-delegation/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot removed the review needed label Sep 1, 2020
@javascript-translate-bot

javascript-translate-bot commented Sep 1, 2020

Copy link
Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot added the changes requested label Sep 1, 2020
@vplentinax

vplentinax commented Sep 1, 2020

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 .

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Sep 1, 2020

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Sep 1, 2020

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Sep 1, 2020 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

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"

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@EzequielCaste

EzequielCaste commented Sep 1, 2020

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Sep 1, 2020

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Sep 1, 2020

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Sep 1, 2020

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vplentinax

vplentinax commented Sep 1, 2020 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@joaquinelio

joaquinelio commented Sep 1, 2020

Copy link
Copy Markdown
Member Author

ya etá roto, no tenia gana de eguir

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

joaquinelio and others added 4 commits September 4, 2020 19:17
@joaquinelio @vplentinax
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
e92bf90
Co-authored-by: Valentina VP <[email protected]>
@joaquinelio @vplentinax
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
82ef753
Co-authored-by: Valentina VP <[email protected]>
@joaquinelio @vplentinax
Update 2-ui/2-events/03-event-delegation/4-behavior-tooltip/task.md
204f06a
Co-authored-by: Valentina VP <[email protected]>
@joaquinelio
Update article.md
91d4874
vplentinax
vplentinax approved these changes Sep 8, 2020
View reviewed changes

@vplentinax vplentinax left a comment

Copy link
Copy Markdown
Contributor

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@javascript-translate-bot javascript-translate-bot added needs +1 and removed changes requested labels Sep 8, 2020
@vplentinax
vplentinax merged commit 0f685c6 into javascript-tutorial:master Sep 20, 2020
@javascript-translate-bot

javascript-translate-bot commented Sep 20, 2020

Copy link
Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@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

Reviewers

@EzequielCaste EzequielCaste EzequielCaste requested changes
@vplentinax vplentinax vplentinax approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

needs +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@joaquinelio @javascript-translate-bot @EzequielCaste @vplentinax
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.