[wp-trac] [WordPress Trac] #7392: Don't create new autosave revision if nothing has changed yet

[WordPress Trac]

#7392: Don't create new autosave revision if nothing has changed yet
---------------------------------------------------+-----------------------
 Reporter:  Otto42                                 |       Owner:  westi
     Type:  defect (bug)                           |      Status:  reopened
 Priority:  normal                                 |   Milestone:  3.6
Component:  Autosave                               |     Version:  2.6
 Severity:  minor                                  |  Resolution:
 Keywords:  has-patch needs-testing autosave-redo  |
---------------------------------------------------+-----------------------

Comment (by adamsilverstein):

 Replying to [comment:45 azaozz]:
 > Was able to reproduce in Chrome (after lots of attempts). The problem is
 with script timing. This should run after TinyMCE is loaded but before
 tinymce.init(). One way to do this is by firing a JS event. Currently we
 init TinyMCE as soon as it loads as we want it to be ready for use as soon
 as possible.
 >
 > Another option is to move that code from autosave.js to the 'wordpress'
 TinyMCE plugin. As `autosaveLast` is a JS global we can set it initially
 to the as-loaded value and then set it to as-saved-from-tinymce when the
 Visual editor loads. Testing a new patch.

 i'm confused about this line:

 editor.onLoad.add( function() {
 (http://core.trac.wordpress.org/browser/trunk/wp-
 includes/js/autosave.js#L8)

 shouldn't that be

 editor.onLoadEditor.add( function() {

 ?

--
Ticket URL: <http://core.trac.wordpress.org/ticket/7392#comment:48>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software