Skip to content

restore "unsaved changes" warning on unload#3478

Merged
ellisonbg merged 2 commits into
ipython:masterfrom
minrk:rewarn
Jul 1, 2013
Merged

restore "unsaved changes" warning on unload#3478
ellisonbg merged 2 commits into
ipython:masterfrom
minrk:rewarn

Conversation

@minrk

@minrk minrk commented Jun 27, 2013

Copy link
Copy Markdown
Member

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

minrk added 2 commits June 28, 2013 15:40
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

ivanov commented Jun 28, 2013

Copy link
Copy Markdown
Member

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

@ellisonbg

Copy link
Copy Markdown
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