Skip to content

Shadow DOM - #386

Merged
joaquinelio merged 3 commits into
javascript-tutorial:masterfrom
joaquinelio:shadows
Oct 7, 2020
Merged

Shadow DOM#386
joaquinelio merged 3 commits into
javascript-tutorial:masterfrom
joaquinelio:shadows

Conversation

@joaquinelio

@joaquineliojoaquinelio commented Sep 22, 2020

Copy link
Copy Markdown
Member

y sombras serán

No pude traducir las herramientas de chrome porque lo tengo en inglés:
More tools > developer tools > (engranaje) > show user agent shadow dom
agregárelo quien pudiere

@javascript-translate-botjavascript-translate-bot mentioned this pull request Sep 22, 2020
Open
vplentinax
vplentinax requested changes Sep 23, 2020
View reviewed changes

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

¿En qué línea está lo que dices?

Comment thread8-web-components/3-shadow-dom/article.md Outdated
Comment thread8-web-components/3-shadow-dom/article.md Outdated
@javascript-translate-bot

javascript-translate-bot commented Sep 23, 2020

Copy link
Copy Markdown

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

@joaquinelio@vplentinax
Apply suggestions from code review
c29ea2e
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
@joaquinelio

joaquinelio commented Sep 23, 2020•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

linea 15
necesitamos habilitar la opción "Show user agent shadow DOM" en "herramientas".

Supongo, que es "herramientas", supongo que es "herramientas de desarrollo"
pero no queria adivinar "Show user agent shadow DOM"

ah, done
=)

@joaquinelio

joaquinelio commented Sep 23, 2020

Copy link
Copy Markdown
MemberAuthor

/done

vplentinax
vplentinax requested changes Oct 7, 2020
View reviewed changes

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

Si, en las herramientas de desarrollo.

Comment thread8-web-components/3-shadow-dom/article.md Outdated
@javascript-translate-bot

javascript-translate-bot commented Oct 7, 2020

Copy link
Copy Markdown

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

@joaquinelio@vplentinax
"herramientas de desarrollo"
ea751c2
Ok.
¿buscaste "Show user agent shadow DOM"?
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
@joaquinelio

joaquinelio commented Oct 7, 2020

Copy link
Copy Markdown
MemberAuthor

Ok.
¿buscaste "Show user agent shadow DOM"?

@vplentinax

vplentinax commented Oct 7, 2020•
edited
Loading

Copy link
Copy Markdown
Contributor

Haz merge, ya que tu eres como el segundo "reviwer" y así avanzamos más.

@joaquinelio

joaquinelio commented Oct 7, 2020

Copy link
Copy Markdown
MemberAuthor

Es que me había entusiasmado ahora que parecía concurrido...

Pero está, bien, el procedimiento lo permite.

@joaquinelio
joaquinelio merged commit c24c603 into javascript-tutorial:masterOct 7, 2020
@javascript-translate-bot

javascript-translate-bot commented Oct 7, 2020

Copy link
Copy Markdown

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

@joaquinelio
joaquinelio deleted the shadows branch October 7, 2020 03:00
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vplentinaxvplentinaxvplentinax requested changes

Assignees

No one assigned

Labels

changes requested

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@joaquinelio@javascript-translate-bot@vplentinax