Skip to content

Walking the DOM#318

Merged
vplentinax merged 24 commits into
javascript-tutorial:masterfrom
Leired7:walking-the-dom
Jul 29, 2020
Merged

Walking the DOM#318
vplentinax merged 24 commits into
javascript-tutorial:masterfrom
Leired7:walking-the-dom

Conversation

@Leired7

@Leired7 Leired7 commented Jul 24, 2020

Copy link
Copy Markdown
Contributor

Artículo, task y soluciones traducido

@CLAassistant

CLAassistant commented Jul 24, 2020

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

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

@joaquinelio joaquinelio left a comment

Copy link
Copy Markdown
Member

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

Comment thread 2-ui/1-document/03-dom-navigation/1-dom-children/solution.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
@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 joaquinelio changed the title Walking the dom Walking the DOM Jul 25, 2020

@maksumi maksumi left a comment

Copy link
Copy Markdown
Collaborator

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

Comment thread 2-ui/1-document/03-dom-navigation/1-dom-children/task.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/3-navigation-links-which-null/task.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
Comment thread 2-ui/1-document/03-dom-navigation/article.md Outdated
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

Leired7 commented Jul 27, 2020

Copy link
Copy Markdown
Contributor Author

/done

@joaquinelio

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

Leired7 commented Jul 28, 2020

Copy link
Copy Markdown
Contributor Author

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

@joaquinelio

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

@joaquinelio joaquinelio left a comment

Copy link
Copy Markdown
Member

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

Leired7 commented Jul 28, 2020

Copy link
Copy Markdown
Contributor Author

Muchas gracias por la ayuda ☺️ @joaquinelio

@maksumi

maksumi commented Jul 28, 2020

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown

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

@Leired7

Leired7 commented Jul 29, 2020

Copy link
Copy Markdown
Contributor Author

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

7 participants