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

"Close and Shut Down Notebook" shuts down without the confirmation dialog #7384

Merged
merged 1 commit into from
May 30, 2024

Conversation

JasonWeill
Copy link
Collaborator

Fixes #7359.

In Notebook 6 and nbclassic, the option "Close and Shut Down Notebook" (no ellipsis) in the File menu shuts the current notebook's kernel down immediately, and closes the browser window/tab. In Notebook 7, we had been using the JupyterLab "Close and Shut Down Notebook…" (with ellipsis) command, which causes a dialog to appear. In the interest of NB6 feature parity and consistency with the File menu's command name, this does not ask for additional confirmation before closing the notebook.

@JasonWeill JasonWeill added the bug label May 29, 2024
@JasonWeill JasonWeill requested a review from jtpio May 29, 2024 18:14
Copy link
Contributor

Binder 👈 Launch a Binder on branch JasonWeill/notebook/close-and-shutdown-cancel

@jtpio jtpio added this to the 7.3.0 milestone May 30, 2024
Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Thanks!

@jtpio jtpio merged commit 433f180 into jupyter:main May 30, 2024
33 of 34 checks passed
@stanwest
Copy link

In the interest of NB6 feature parity and consistency with the File menu's command name, this does not ask for additional confirmation before closing the notebook.

Thanks for addressing the surprising behavior of the dialog. I've opened the above feature request to restore the confirmation dialog. From the perspective of a user, I'm not very concerned with consistency with Notebook 6, but I am very concerned with inadvertently losing the work I have in the kernel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cancelling "Close and Shut Down Notebook" closes tab anyway
3 participants