Skip to content

Shadow DOM slots, composition #381

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

Merged
merged 2 commits into from
Oct 24, 2022

Conversation

kolumb
Copy link
Contributor

@kolumb kolumb commented Aug 14, 2022

No description provided.

Copy link
Collaborator

@dolgachio dolgachio left a comment

Choose a reason for hiding this comment

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

Дякую, за чудовий переклад! Вибачаюсь, що вам довелося так довго чекати :)
Подивіться, будь ласка, мої коментарі і напишіть мені, якщо з чимось не згодні.

@@ -51,45 +51,45 @@ customElements.define('user-card', class extends HTMLElement {
</script>

<user-card>
<span *!*slot="username"*/!*>John Smith</span>
<span *!*slot="username"*/!*>Тарас Мельник</span>
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@javascript-translate-bot
Copy link
Contributor

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

@dolgachio dolgachio removed the in progress Translation in progress label Oct 21, 2022
@kolumb
Copy link
Contributor Author

kolumb commented Oct 21, 2022

/done

@dolgachio
Copy link
Collaborator

@all-contributors add @kolumb for translation

@allcontributors
Copy link
Contributor

@stas-dolgachov

@kolumb already contributed before to translation

@dolgachio dolgachio merged commit ea0420f into javascript-tutorial:master Oct 24, 2022
@javascript-translate-bot
Copy link
Contributor

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

@kolumb kolumb deleted the shadow-dom-slots branch October 24, 2022 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants