Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Добавление раздела: WeakRef & FinalizationRegistry#1899

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Alexandre887 merged 6 commits intojavascript-tutorial:masterfromWOLFRIEND:master
Oct 31, 2023

Conversation

WOLFRIEND
Copy link
Contributor

Описание

Добавление информации о таких возможностях какWeakRef &FinalizationRegistry.
Обсуждения и утверждение данного предложения находятся по ссылке:
#1898

Ссылки

Связанные Issue

@javascript-translate-botjavascript-translate-bot added the review neededReview needed, please approve or request changes labelOct 12, 2023
@CLAassistant
Copy link

CLAassistant commentedOct 12, 2023
edited
Loading

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@Alexandre887Alexandre887 left a comment
edited
Loading

Choose a reason for hiding this comment

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

Добрый вечер.

Не пугайтесь большого количества правок — в основном это suggested changes, которые можно принять в один клик.

Также:

  • Я оставил комментарии к некоторым строкам кода, т.к не везде понятна ваша мысль.
  • Я оставил комментарии к некоторым частям текста, где есть тавтология и/или логические ошибки. Также где-то определённые выражения звучат не совсем по-русски, так что их нужно перефразировать.
  • Были ошибки, которые распространяются на весь материал статьи. К примеру, неверное написание определённых слов. В таком случае я не искал и не исправлял эти ошибки во всём тексте, просто оставил комментарий к одному подобному примеру, дальше вы должны будете сами их исправить.

Обязательно сначала примите suggested changes, а только потом рассматривайте комментарии. Иногда я оставлял и suggested changes, и комментарий в одной правке, так чтовнимательно всё прочтите.

Если мои замечания где-то не совсем понятны, или вы хотите их оспорить, — в Github также можно оставлять к ним комментарии, я получу уведомление и мы обсудим эти вопросы.

В техническую часть материала я пока глубоко не вдавался, сфокусировался на тексте и коде.

В раздел «Типы данных», на мой взгляд, статью будет помещать неправильно. Это очень узконаправленная тема, которая на практикене затрагивает 90% всех программистов. Многие спокойно пишут код, даже не зная, что такие возможности были введены в язык. Оно и к лучшему, т.к даже в mdn приведён перечень причин, по которым лучше избегать использования структур, о которых идет речь в статье.
Поэтому, лучше всего будет добавить статью в раздел «Разное», в конец первой части учебника.

В начале статьи можем добавить предостережение, указать в нём причины, которые я привёл выше. Также следует добавить информацию о том, что для прочтения необходимы базовые знания вDOM, т.к в статье есть примеры с его использованием.
Но, сейчас это не столь важно. Сосредоточьтесь на рассмотрении моих правок.

WOLFRIEND reacted with thumbs up emoji
@javascript-translate-botjavascript-translate-bot added changes requestedWaiting for changes and /done from PR author and removed review neededReview needed, please approve or request changes labelsOct 22, 2023
@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 👻

Co-authored-by: Aleksandras <alex30955@aol.com>
@WOLFRIEND
Copy link
ContributorAuthor

Спасибо большое за правки и предложение.
Я ознакомился и принял ваши suggested changes.
Теперь начинаю работу над вашими комментариями.

@WOLFRIEND
Copy link
ContributorAuthor

/done

@javascript-translate-botjavascript-translate-bot added review neededReview needed, please approve or request changes and removed changes requestedWaiting for changes and /done from PR author labelsOct 25, 2023
@WOLFRIEND
Copy link
ContributorAuthor

Добрый день,

Я внёс изменения в соответствии с вашими правками:

  1. Принял ваши рекомендации.
  2. Исправил перевод текста в необходимых местах (картинки, примеры).
  3. Устранил ошибки, исправил непонятные описания и объяснения в соответствии с вашими комментариями.
  4. Перенёс статью в раздел, который вы предложили.
  5. Добавил необходимые предупреждения.

Я старался максимально следовать вашим правкам и комментариям. Пожалуйста, оцените результат и скажите, нужны ли дополнительные правки к материалу? Спасибо за вашу помощь!

@javascript-translate-botjavascript-translate-bot added changes requestedWaiting for changes and /done from PR author and removed review neededReview needed, please approve or request changes labelsOct 29, 2023
@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 👻

@WOLFRIEND
Copy link
ContributorAuthor

/done

@javascript-translate-botjavascript-translate-bot added review neededReview needed, please approve or request changes and removed changes requestedWaiting for changes and /done from PR author labelsOct 30, 2023
@WOLFRIEND
Copy link
ContributorAuthor

Добрый день,

Я внёс изменения в соответствии с вашими правками и ответил в комментариях.

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

@Alexandre887Alexandre887Alexandre887 approved these changes

Assignees
No one assigned
Labels
needs +1One more review needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@WOLFRIEND@CLAassistant@javascript-translate-bot@Alexandre887

[8]ページ先頭

©2009-2025 Movatter.jp