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

Дополняет описание options в доке .addEventListener() #5647

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

vladislav149
Copy link
Contributor

Описание

Добавил ключ signal в options и краткий пример кода.

Чек-лист

  • Текст оформлен согласно руководству по стилю
  • Ссылки на внутренние материалы начинаются со слеша и заканчиваются слэшем либо якорем на заголовок (/css/color/, /tools/json/, /tools/gulp/#kak-ponyat)
  • Ссылки на картинки, видео и демки относительные (images/example.png, demos/example/, ../demos/example/)

@github-actions github-actions bot added js Контент по JavaScript дока Справочный материал labels Jan 18, 2025
@vitya-ne vitya-ne added the улучшение Доработка существующего label Jan 18, 2025
@vitya-ne
Copy link
Contributor

@vladislav149 Спасибо, за дополнение.
Я предлагая также добавить абзац с примером отписки с помощью AbortSignal в доку .removeEventListener()
Как тебе ?

@vladislav149
Copy link
Contributor Author

vladislav149 commented Jan 19, 2025

@vladislav149 Спасибо, за дополнение. Я предлагая также добавить абзац с примером отписки с помощью AbortSignal в доку .removeEventListener() Как тебе ?

Да, я хотел вот такой пример, как думаешь, норм?

const abortController = new AbortController()
element1.addEventListener('click', () => console.log('Подписка активна'), {
  signal: abortController.signal,
})
element2.addEventListener('click', () => {
  abortController.abort()
  console.log('Подписка отменена')
})

@vitya-ne
Copy link
Contributor

Да, я хотел вот такой пример, как думаешь, норм?

const abortController = new AbortController()
element1.addEventListener('click', () => console.log('Подписка активна'), {
  signal: abortController.signal,
})
element2.addEventListener('click', () => {
  abortController.abort()
  console.log('Подписка отменена')
})

Да, отлично. И возможно стоит указать, когда использование AbortController оправдано.

@vitya-ne vitya-ne changed the title Добавлен ключ signal в options Дополняет описание options в доке .addEventListener() Jan 19, 2025
@vitya-ne
Copy link
Contributor

@vladislav149 Пожалуйста, добавь себя в список авторов.
Вот дока как это сделать: https://github.com/doka-guide/content/blob/main/docs/people.md

Copy link
Member

@Inventoris Inventoris left a comment

Choose a reason for hiding this comment

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

Привет, классно, что добавил! Спасибо 🙌

Предлагаю рассказать чуть больше про такой способ отписки от события, там есть много интересного:

js/element-addeventlistener/index.md Outdated Show resolved Hide resolved
@vladislav149 vladislav149 force-pushed the AbortController-in-addEventListener branch from e75f6bb to 8167ded Compare January 26, 2025 16:56
Copy link
Member

@Inventoris Inventoris left a comment

Choose a reason for hiding this comment

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

Влад, круто! Мне очень нравится, что ты ёмко добавил столько полезной инфы про AbortController в раздел "На практике", спасибо!

Оставил несколько комментариев с мелочами, посмотри ещё пожалуйста.

js/element-addeventlistener/index.md Outdated Show resolved Hide resolved
js/element-addeventlistener/index.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
js/element-addeventlistener/practice/vladislav149.md Outdated Show resolved Hide resolved
Co-authored-by: Alexey Nikitchenko <[email protected]>
Copy link

Превью контента из 8c8d845 опубликовано.

@vladislav149
Copy link
Contributor Author

Влад, круто! Мне очень нравится, что ты ёмко добавил столько полезной инфы про AbortController в раздел "На практике", спасибо!

Оставил несколько комментариев с мелочами, посмотри ещё пожалуйста.

Спасибо! Принял)

Copy link
Member

@Inventoris Inventoris left a comment

Choose a reason for hiding this comment

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

Апрув и точка.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
js Контент по JavaScript дока Справочный материал улучшение Доработка существующего
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants