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 — Introduction to browser events by maksumi · Pull Request #392 · javascript-tutorial/es.javascript.info

Conversation

@maksumi

@maksumi maksumi commented Sep 28, 2020

Copy link
Copy Markdown
Collaborator

No description provided.

Sorry, something went wrong.

Uh oh!

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

All reactions
Maksumi Muralami added 4 commits August 16, 2020 01:16
Maksumi Muralami
Merge branch 'master' of https://github.com/javascript-tutorial/es.ja…
536f942
…vascript.info
Maksumi Muralami
Traducción del artículo
8029539
Maksumi Muralami
1er parte de las tareas traducidas
2b61275
Maksumi Muralami
Traducción del articulo y las tareas
62414a2
@javascript-translate-bot javascript-translate-bot added the review needed label Sep 28, 2020
@javascript-translate-bot
javascript-translate-bot requested a review from a team September 28, 2020 20:24
@javascript-translate-bot javascript-translate-bot mentioned this pull request Sep 28, 2020
Spanish Translation Progress #17
Open
joaquinelio
joaquinelio requested changes Sep 29, 2020
View reviewed changes

@joaquinelio joaquinelio left a comment •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown
Member

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.

algunas cosas.

algunas son dudas.

No me gusta "luce" como verse o mostrarse (deberia ponerse "se luce como" que es peor), no es buen español (y encima lucir tiene connotacion de exhibicion, no solo mostrar), pero ahora todos los periodistas lo traducen así, mal. Asi que a nadie va a molestar y lo va a entender rapido asi que no lo corregi

Sorry, something went wrong.

Uh oh!

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

All reactions
Comment thread 2-ui/2-events/01-introduction-browser-events/01-hide-other/task.md Resolved

Uh oh!

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

Comment thread 2-ui/2-events/01-introduction-browser-events/02-hide-self-onclick/task.md Resolved

Uh oh!

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

Comment thread 2-ui/2-events/01-introduction-browser-events/04-move-ball-field/solution.md
No podemos usar `position:fixed` para ello, porque al desplazar la página se movería el balón del campo.

So we should use `position:absolute` and, to make the positioning really solid, make `field` itself positioned.
Así que deberíamos usar `position:absolute` y, para que el posicionamiento sea realmente sólido, hacer que `field` sea posicione a sí mismo.

@joaquinelio joaquinelio Sep 29, 2020

Copy link
Copy Markdown
Member

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.

entendemos "solido" ?

en ingles "solid" es una persona confiable, ¿el ruso lo esta usando asi? seria medio rebuscado

la cosa que no "posicionamiento solido" que se refiere a coordenadas relativass al contenido no al browser
aunque quizaas sea equivalente a decir :absolute y np lo entiendo...

ok no importa

Sorry, something went wrong.

Uh oh!

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

All reactions
Comment thread 2-ui/2-events/01-introduction-browser-events/06-hide-message/solution.md Resolved

Uh oh!

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

Comment thread 2-ui/2-events/01-introduction-browser-events/article.md Resolved

Uh oh!

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

Comment thread 2-ui/2-events/01-introduction-browser-events/article.md Resolved

Uh oh!

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

Comment thread 2-ui/2-events/01-introduction-browser-events/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 29, 2020
@javascript-translate-bot

javascript-translate-bot commented Sep 29, 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 29, 2020
Maksumi Murakami @joaquinelio
Realizado
cd05e8e
/done

Co-authored-by: joaquinelio <[email protected]>
@maksumi

maksumi commented Sep 30, 2020

Copy link
Copy Markdown
Collaborator 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 Sep 30, 2020
@javascript-translate-bot
javascript-translate-bot requested a review from a team September 30, 2020 18:24
@maksumi

maksumi commented Sep 30, 2020

Copy link
Copy Markdown
Collaborator Author

Respecto a lo de sólido. Entiendo la confusión pero una vez dominado se entiende que se refiere en relatividad al contenido. Po cierto, mu buen ojo con esos detalles, gracias como siempre

All reactions

Sorry, something went wrong.

Uh oh!

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

joaquinelio
joaquinelio approved these changes Sep 30, 2020
View reviewed changes
@javascript-translate-bot javascript-translate-bot added needs +1 and removed review needed labels Sep 30, 2020
vplentinax
vplentinax reviewed Oct 1, 2020
View reviewed changes
Comment thread 2-ui/2-events/01-introduction-browser-events/03-which-handlers-run/task.md
---

# Which handlers run?
# ¿Qué handlers se ejecutan?

@vplentinax vplentinax Oct 1, 2020

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.

Correcto, pero tengo una duda ¿no hemos traducido handler en anteriores artículos? Creo que es mejor mantener esa consistencia. No importa si se traduce como manejador o controlador, pero creo que es mejor traducirlo.

Sorry, something went wrong.

Uh oh!

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

All reactions
vplentinax
vplentinax reviewed Oct 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.

Todo correcto, solo quiero eliminar la duda sobre "handler" que veo que no se tradujo en ningún momento. ¿No creen que es mejor traducirlo? Sea manejador o controlador, pero traducido.
@joaquinelio @maksumi

Sorry, something went wrong.

Uh oh!

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

All reactions
maksumi
maksumi commented Oct 1, 2020
View reviewed changes
Comment thread 2-ui/2-events/01-introduction-browser-events/article.md Resolved

Uh oh!

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

Comment thread 2-ui/2-events/01-introduction-browser-events/article.md Resolved

Uh oh!

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

Maksumi Murakami added 2 commits October 1, 2020 11:28
Maksumi Murakami
Update 2-ui/2-events/01-introduction-browser-events/article.md
721c0cd
Maksumi Murakami
Update 2-ui/2-events/01-introduction-browser-events/article.md
7c0d1ae
@maksumi

maksumi commented Oct 1, 2020

Copy link
Copy Markdown
Collaborator Author

Hola @vplentinax
Tu duda se encuentra resuelta en la linea 29 y 31 del article. Había traducido por manejadores pero me convence más controladores y lo actualice así

All reactions

Sorry, something went wrong.

Uh oh!

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

vplentinax
vplentinax approved these changes Oct 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.

Ok!

Sorry, something went wrong.

Uh oh!

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

All reactions
@vplentinax vplentinax added ready to merge and removed needs +1 labels Oct 1, 2020
@joaquinelio

joaquinelio commented Oct 2, 2020 •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown
Member

Ja ja, lo que pasa es que @maksumi me hizo trampa,
esperó a que aprobara y despues lo cambió

Muchass cosas no nos ponemos de acuerdo, por eso el respeto a lo que decide el que traduce

En la ssync que no termine porque sigo por el piso,
un articulo @vplentinax usa controladorr y cuando tuve que agregar algo use controlador,
si bien no es bueno tener desacuerdos en conceptos
peor es la falta de consistencia dentro de un mimo articulo

no traducirlo era una opcion, TODO en js es handlers, acá y alla,
el primer proyecto que hice decidí en lugar de usar usar interfaces y eventos para un CRUD,
use filosofia JS, callbacks por todos lados, handlers.
en este caso no me gusta usar la missma palabra para diferentes conceptos
JS no tiene controllers, pero handler es un método, una funcion mundana que paseamos
controller suele ser un objeto mas complejo y poderoso, o una interfaz
¿seguro? no, me hubiera gustado la opinion de alguien mas veterano

lo que no podemos es frenar todo por buscar lo perfecto
o hacer asamblea en cada articulo

quedan 3 art por reclamar, 11 reclamados de julio, apenas dos meses) pero podria liberar algunos
y esta la sincro y retrducciones, debo aun el issue con faltantes to fix fuera del issue17,
falta...

All reactions

Sorry, something went wrong.

Uh oh!

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

@joaquinelio

joaquinelio commented Oct 2, 2020

Copy link
Copy Markdown
Member

@maksumi

molessta esto aca che

crei que habias cambiado texto ¿vas a modificarlo? si decidiss cambiar solo esa palabra por todos lados no problem, sigue aprobado

pero que quede esto aca ocioso... faltando solo 1 para quebrar el numero 30...

30 faltantes para terminar. 10 aun por reclamar

el unico que deje reclamado en julio es el fatidico proxy, je

All reactions

Sorry, something went wrong.

Uh oh!

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

@maksumi

maksumi commented Oct 2, 2020

Copy link
Copy Markdown
Collaborator Author

Estoy trabajando en el fatidico Proxy de hecho, pero ese lo llevo a poco. Gracias por avisar. Ya estaba revisando cuál tomar
@joaquinelio

All reactions

Sorry, something went wrong.

Uh oh!

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

@maksumi
maksumi merged commit 0df72fc into javascript-tutorial:master Oct 2, 2020
@javascript-translate-bot

javascript-translate-bot commented Oct 2, 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joaquinelio joaquinelio joaquinelio approved these changes
@vplentinax vplentinax vplentinax approved these changes

Uh oh!

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

Assignees

No one assigned

Labels

ready to merge

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

@maksumi @javascript-translate-bot @joaquinelio @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.