Skip to content

Добавляет раздел про обработку событий в доку Element #5990

Open
lira-bazh wants to merge 23 commits into
doka-guide:mainfrom
lira-bazh:doka/element-event
Open

Добавляет раздел про обработку событий в доку Element #5990
lira-bazh wants to merge 23 commits into
doka-guide:mainfrom
lira-bazh:doka/element-event

Conversation

@lira-bazh
Copy link
Copy Markdown
Contributor

@lira-bazh lira-bazh commented May 2, 2026

Описание

2026-05-02_15-36-39

По мотивам обсуждения в пр для blur

Чек-лист

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

@github-actions github-actions Bot added js Контент по JavaScript дока Справочный материал labels May 2, 2026
Comment thread js/element/index.md Outdated
Comment thread js/element/index.md Outdated
Comment thread js/element/index.md Outdated
Comment thread js/element/index.md Outdated
Comment thread js/element/index.md
Comment thread js/element/index.md Outdated
Comment thread js/element/index.md Outdated
Comment thread js/event-unload/index.md Outdated
@vitya-ne
Copy link
Copy Markdown
Contributor

vitya-ne commented May 2, 2026

@lira-bazh привет,
Спасибо за объёмное дополнение.
Я добавил несколько коментов.

@vitya-ne vitya-ne changed the title В доку Element добавлен раздел про обработку событий Добавляет раздел про обработку событий в доку Element May 2, 2026
lira-bazh and others added 2 commits May 3, 2026 17:03
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
@lira-bazh lira-bazh requested review from vitya-ne May 4, 2026 14:40
Comment thread js/element/index.md Outdated
Comment thread js/element/index.md Outdated
lira-bazh and others added 2 commits May 4, 2026 20:14
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Comment thread js/element/index.md
Comment thread js/element-addeventlistener/index.md Outdated
Comment thread js/element-addeventlistener/index.md Outdated
lira-bazh and others added 3 commits May 13, 2026 11:28
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Comment thread js/element-addeventlistener/index.md Outdated
Comment thread js/element-removeeventlistener/index.md Outdated
Comment thread js/element-removeeventlistener/index.md Outdated
Comment thread js/element/index.md Outdated
Comment thread js/element-addeventlistener/index.md Outdated
lira-bazh and others added 5 commits May 14, 2026 14:42
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
Comment thread js/element-addeventlistener/index.md Outdated
Comment thread js/element-addeventlistener/index.md
Comment thread js/element-addeventlistener/index.md Outdated
@lira-bazh
Copy link
Copy Markdown
Contributor Author

Заметила, что в доке по removeEventListener пропустила ещё несколько упоминаний того, что параметры при удалении должны полностью совпадать. Поправила

@lira-bazh lira-bazh requested a review from vitya-ne May 14, 2026 17:16
Comment thread js/element-addeventlistener/index.md Outdated
Comment thread js/element-addeventlistener/index.md Outdated
Copy link
Copy Markdown
Contributor

@vitya-ne vitya-ne left a comment

Choose a reason for hiding this comment

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

Заметил ещё пару, мелочей.
Есть еще сомнение насчёт недостатка:

если забыть удалить обработчики, есть риск утечки памяти;

по-моему, это и для встроенных обработчиков верно. Может стоит это убрать ?

lira-bazh and others added 2 commits May 15, 2026 15:31
Co-authored-by: Vitya <9317613+vitya-ne@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

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

@vitya-ne
Copy link
Copy Markdown
Contributor

@lira-bazh Спасибо за труд и внимание к мелочам

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.

3 participants