-
Notifications
You must be signed in to change notification settings - Fork 649
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Класс. Спасибо. Давай чуть подправим код и помержим
Превью контента из 0acea4c опубликовано. |
Можете вызвать метод, только если окно открыто через [`window.open()`](/js/window-open/) или это **окно верхнего уровня** без **записей в истории**. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
привет,
Можете вызвать метод, только если..
А что будет если попытаться закрыть текущее окно ? Ошибка ведь не произойдет.
может стоит немного изменить фразу, что бы дать понять что метод просто не выполнит закрытие если ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
согласна )
|
||
💡 Используйте `window.close()`, когда и правда нужно закрыть окно. Пользователи в большинстве случаев не ожидают закрытия окон без их согласия. | ||
|
||
💡 Убедитесь, что окно, которое пытаетесь закрыть, открыто с помощью скрипта. Если это не так, браузер проигнорирует вызов `window.close()`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
а как проверить что окно открыто с помощью скрипта ?
There was a problem hiding this comment.
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/) или это **окно верхнего уровня** без **записей в истории**. | ||
|
There was a problem hiding this comment.
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()`. |
There was a problem hiding this comment.
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()` не работает из-за настроек браузера, которые ограничивают или блокируют возможность программного закрытия окон. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай добавим конкретный пример (а то не нашла об этом в спеке)
Описание
Closes #315.
Чек-лист
/css/color/
,/tools/json/
,/tools/gulp/#kak-ponyat
)images/example.png
,demos/example/
,../demos/example/
)