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

Walking the DOM #318

Merged
merged 24 commits into from
Jul 29, 2020
Merged

Conversation

Leired7
Copy link
Contributor

@Leired7 Leired7 commented Jul 24, 2020

Artículo, task y soluciones traducido

@CLAassistant
Copy link

CLAassistant commented Jul 24, 2020

CLA assistant check
All committers have signed the CLA.

@joaquinelio joaquinelio self-requested a review July 24, 2020 23:23
Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

Buena adición al repo,

algún typo,
algún cambio de construcción opinable (que podes ignorar y marcar resuelto)

Esstuve tentado a pedir algo más neutral de "una pega"
pero, argentino, la entendí, es perfecta.

Vamos segundo reviewer, ven a reviegüesear...

EDIT: nota, Don Segundo Reviewer: el bot no lo reconoció, lo agrego a mano. Arreglé el nombre supongo que con DOM mayúsculas lo verá cuando merge. Verificar...

2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/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 joaquinelio changed the title Walking the dom Walking the DOM Jul 25, 2020
Copy link
Collaborator

@maksumi maksumi left a comment

Choose a reason for hiding this comment

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

Muy buena traducción. Solo arreglar unos tipos y considerar algunas sugerencias para mantener un lenguaje simple en la traducción

2-ui/1-document/03-dom-navigation/1-dom-children/task.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
2-ui/1-document/03-dom-navigation/article.md Outdated Show resolved Hide resolved
Leired7 and others added 13 commits July 27, 2020 07:46
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
…l/task.md

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@Leired7 Leired7 closed this Jul 27, 2020
@Leired7 Leired7 reopened this Jul 27, 2020
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@Leired7 Leired7 closed this Jul 27, 2020
@Leired7 Leired7 reopened this Jul 27, 2020
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@Leired7 Leired7 requested a review from joaquinelio July 27, 2020 06:12
@Leired7
Copy link
Contributor Author

Leired7 commented Jul 27, 2020

/done

@joaquinelio
Copy link
Member

@Leired7 ... ¿closed?
Supongo que ya estaba listo para última mirada e integrarlo,
¿puedo suponer que fue un error de "dóndehagoclicahora"? je je tuve muchos de esos cuando empecé con la traducción...

@Leired7
Copy link
Contributor Author

Leired7 commented Jul 28, 2020

Síiii @joaquinelio!!!! exactamente paso eso! ¿Dónde "tengoqueclickarahora"?

@joaquinelio
Copy link
Member

No problem...
El autor puede cerrarlo para modificar sin que nadie lo moleste y reabrirlo o dejarlo cerrado y hacer uno nuevo, no pasa nada.
Acá abajo, "Reopen and comment"...
Pero como dijiste que fuer por error, me tenté y abro y reviso...

@joaquinelio joaquinelio reopened this Jul 28, 2020
joaquinelio and others added 2 commits July 28, 2020 12:12
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

No es perfecto, (sutilezas intraducibles)
pero menos será si hacemos de la redacción una asamblea.
Apliqué un par de sug de maksumi, dejé un par afuera, y otra que hice y no supe mejorar.
Funciona como está.

@Leired7
Copy link
Contributor Author

Leired7 commented Jul 28, 2020

Muchas gracias por la ayuda ☺️ @joaquinelio

@maksumi
Copy link
Collaborator

maksumi commented Jul 28, 2020

Merge it! Ya llegará el momento de detallar las sutilezas

@vplentinax vplentinax merged commit 9002711 into javascript-tutorial:master Jul 29, 2020
@javascript-translate-bot

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

@Leired7
Copy link
Contributor Author

Leired7 commented Jul 29, 2020

¡Qué alegría! jejej Gracias!!!

A por el siguiente....

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

Successfully merging this pull request may close these issues.

None yet

7 participants