Looking pretty good! So far only one issue.

I am not running Tinymce and am getting the following error on line 37 of autosave.js.

tinyMCE is not defined
saveForm()
tinyMCE.selectedInstance.selection.moveToBookmark(editorBookmark);

Following the js error the form will not autosave again.

CommentFileSizeAuthor
#5 autosave-without-tinymce.diff1.48 KBAnonymous (not verified)
#1 autosave.patch625 bytesDavid Lesieur

Comments

David Lesieur’s picture

David Lesieur commented
Priority: Normal » Critical
StatusFileSize
new autosave.patch625 bytes

Because of this bug, autosave doesn't work without TinyMCE. This patch fixes this.

David Lesieur’s picture

David Lesieur commented
Status: Active » Needs review
liquidcms’s picture

liquidcms commented

thanks for the patch Dave; this is now included in latest rev (5.x-3.2)

liquidcms’s picture

liquidcms commented
Status: Needs review » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Needs review
StatusFileSize
new autosave-without-tinymce.diff1.48 KB

Actually, I still had problems running without tinymce. It would complain that tinyMCE.selectedInstance was null. I've got a fix.