Adding a new 'save' event

[Garry Yao]

Many users are looking for a easy way to override the default save button behavior, i.e. ajax post. A new 'save' event could be fired by save plugin on editor instance level to make this customization happen.

[Dekel]

patch for #4507

[Dekel]

patch attached.

usage (In the configuration file) :

/*
Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved.
For licensing, see LICENSE.html or http://ckeditor.com/license
*/

CKEDITOR.editorConfig = function( config )
{
	// Define changes to default configuration here. For example:
	// config.language = 'fr';
	// config.uiColor = '#AADC6E';
	
	config.saveFunction = function(data) {
		alert(data)
	}
};

[Frederico Caldeira Knabben]

We should simply use our standard event system here on the CKEDITOR.editor class.

[Scott Bronson]

fredck, true, but right now the button is grayed out if the editor isn't in a form. How would you enable it so the callback could fire?

It would be OK by me if the save button was always enabled and didn't check to see if the editor was in a form or not... not sure if this has legacy issues though.

[Frederico Caldeira Knabben]

Milestone CKEditor 3.x deleted

[Garry Yao]

I don't see there a necessary for this into core now, as an ajax save could be instead provided by a custom save plugin (resolves the form sensitive state problem also).

[Frederico Caldeira Knabben]

@bronson, you're right. At this point, maybe the configuration option would do it well.

@garry.yao, the idea is making it deadly simple, so it's definitely a valid need.

[Piotrek Koszuliński]

Fixed in 4.2: ​http://docs.ckeditor.com/#!/api/CKEDITOR.editor-event-save