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

Добавляет доку про window.close() #5527

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

Conversation

ra1nbow1
Copy link
Member

@ra1nbow1 ra1nbow1 commented Oct 13, 2024

Описание

Closes #315.

Чек-лист

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

@github-actions github-actions bot added js Контент по JavaScript дока Справочный материал labels Oct 13, 2024
js/window-close/index.md Outdated Show resolved Hide resolved
Copy link
Member

@HellSquirrel HellSquirrel left a comment

Choose a reason for hiding this comment

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

Класс. Спасибо. Давай чуть подправим код и помержим

js/window-close/index.md Outdated Show resolved Hide resolved
Copy link

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

Comment on lines +32 to +33
Можете вызвать метод, только если окно открыто через [`window.open()`](/js/window-open/) или это **окно верхнего уровня** без **записей в истории**.

Copy link
Contributor

Choose a reason for hiding this comment

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

привет,

Можете вызвать метод, только если..

А что будет если попытаться закрыть текущее окно ? Ошибка ведь не произойдет.
может стоит немного изменить фразу, что бы дать понять что метод просто не выполнит закрытие если ...

Copy link
Member

Choose a reason for hiding this comment

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

согласна )


💡 Используйте `window.close()`, когда и правда нужно закрыть окно. Пользователи в большинстве случаев не ожидают закрытия окон без их согласия.

💡 Убедитесь, что окно, которое пытаетесь закрыть, открыто с помощью скрипта. Если это не так, браузер проигнорирует вызов `window.close()`.
Copy link
Contributor

Choose a reason for hiding this comment

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

а как проверить что окно открыто с помощью скрипта ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Создать его и быть уверенным в этом

Comment on lines +32 to +33
Можете вызвать метод, только если окно открыто через [`window.open()`](/js/window-open/) или это **окно верхнего уровня** без **записей в истории**.

Copy link
Member

Choose a reason for hiding this comment

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

согласна )

Метод можно вызвать только если окно было открыто через [`window.open()`](/js/window-open/) или если это окно верхнего уровня без записей в истории.
Можете вызвать метод, только если окно открыто через [`window.open()`](/js/window-open/) или это **окно верхнего уровня** без **записей в истории**.

**Окно верхнего уровня** — это основное окно браузера, которое было открыто пользователем вручную, например, обычная вкладка или окно приложения. Браузеры часто запрещают закрытие таких окон с помощью `window.close()`, если они не были открыты через `window.open()`.
Copy link
Member

Choose a reason for hiding this comment

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

Давай это поднимем повыше. Сначала терминология и потом описание )


В современных браузерах вызов `window.close()` в окне, которое не было открыто с помощью JavaScript, не сработает. Например, с помощью этого метода невозможно закрыть вкладку или главное окно (страницу).

Иногда метод `window.close()` не работает из-за настроек браузера, которые ограничивают или блокируют возможность программного закрытия окон.
Copy link
Member

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.

Интерфейс командной строки
4 participants