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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

David Lesieur’s picture

David Lesieur CreditAttribution: David Lesieur commented
Priority: Normal » Critical
FileSize
autosave.patch625 bytes

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

David Lesieur’s picture

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

liquidcms CreditAttribution: liquidcms commented

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

liquidcms’s picture

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

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Needs review
FileSize
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.