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

restore "unsaved changes" warning on unload #3478

Merged
merged 2 commits into from Jul 1, 2013
Merged

Conversation

minrk
Copy link
Member

@minrk minrk commented Jun 27, 2013

Autosave is triggered at this event, but it seems to fail sometimes (perhaps when it's too slow?).

This restores the "beforeunload" warning message, informing the user that unsaved changes may be lost.
It still triggers an autosave (assuming autosave is enabled), in which case the message is "Autosave in progress".

closes #3475

Autosave is triggered at this event, but it seems to fail sometimes (perhaps when it's too slow?).

closes ipython#3475
mention pending autosave, if applicable
@ivanov
Copy link
Member

ivanov commented Jun 28, 2013

thanks, I'm going to test this out and report back

@ellisonbg
Copy link
Member

This works as expected and is the final step in getting all of the dirty/autosave stuff right. Making progress...

ellisonbg added a commit that referenced this pull request Jul 1, 2013
restore "unsaved changes" warning on unload
@ellisonbg ellisonbg merged commit 1359114 into ipython:master Jul 1, 2013
@minrk minrk deleted the rewarn branch March 31, 2014 23:36
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
restore "unsaved changes" warning on unload
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

no warning when leaving/closing notebook on master without saved changes
3 participants