id summary reporter owner description type status priority milestone component version resolution keywords cc
11282 Documentation on Widgets (Part 2) has an error. Dennis Dewey Anna Tomanek "'''On this page :'''
[http://docs.ckeditor.com/#!/guide/widget_sdk_tutorial_2]
'''This code block :'''
{{{
CKEDITOR.plugins.add( 'simplebox', {
requires: 'widget',
icons: 'simplebox',
init: function( editor ) {
CKEDITOR.dialog.add( 'simplebox', this.path + 'dialogs/simplebox.js' );
editor.widgets.add( 'simplebox', {
button: 'Create a simple box',
template:
'
',
editables: {
title: {
selector: '.simplebox-title',
allowedContent: 'br strong em'
},
content: {
selector: '.simplebox-content',
allowedContent: 'p br ul ol li strong em'
}
},
allowedContent:
'div(!simplebox,align-left,align-right,align-center){width};' +
'div(!simplebox-content); h2(!simplebox-title)',
requiredContent: 'div(simplebox)',
dialog: 'simplebox',
upcast: function( element ) {
return element.name == 'div' && element.hasClass( 'simplebox' );
},
init: function() {
var width = this.element.getStyle( 'width' );
if ( with )
this.setData( 'width', width );
if ( this.element.hasClass( 'align-left' ) )
this.setData( 'align', 'left' );
if ( this.element.hasClass( 'align-right' ) )
this.setData( 'align', 'right' );
if ( this.element.hasClass( 'align-center' ) )
this.setData( 'align', 'center' );
},
data: function() {
if ( this.data.width == '' )
this.element.removeStyle( 'width' );
else
this.element.setStyle( 'width', this.data.width );
this.element.removeClass( 'align-left' );
this.element.removeClass( 'align-right' );
this.element.removeClass( 'align-center' );
if ( this.data.align )
this.element.addClass( 'align-' + this.data.align );
}
} );
}
} );
}}}
'''Needs to be replaced with :'''
{{{
CKEDITOR.plugins.add( 'simplebox', {
requires: 'widget',
icons: 'simplebox',
init: function( editor ) {
CKEDITOR.dialog.add( 'simplebox', this.path + 'dialogs/simplebox.js' );
editor.widgets.add( 'simplebox', {
button: 'Create a simple box',
template:
'',
editables: {
title: {
selector: '.simplebox-title',
allowedContent: 'br strong em'
},
content: {
selector: '.simplebox-content',
allowedContent: 'p br ul ol li strong em'
}
},
allowedContent:
'div(!simplebox,align-left,align-right,align-center){width};' +
'div(!simplebox-content); h2(!simplebox-title)',
requiredContent: 'div(simplebox)',
dialog: 'simplebox',
upcast: function( element ) {
return element.name == 'div' && element.hasClass( 'simplebox' );
},
init: function() {
var width = this.element.getStyle( 'width' );
if ( width )
this.setData( 'width', width );
if ( this.element.hasClass( 'align-left' ) )
this.setData( 'align', 'left' );
if ( this.element.hasClass( 'align-right' ) )
this.setData( 'align', 'right' );
if ( this.element.hasClass( 'align-center' ) )
this.setData( 'align', 'center' );
},
data: function() {
if ( this.data.width == '' )
this.element.removeStyle( 'width' );
else
this.element.setStyle( 'width', this.data.width );
this.element.removeClass( 'align-left' );
this.element.removeClass( 'align-right' );
this.element.removeClass( 'align-center' );
if ( this.data.align )
this.element.addClass( 'align-' + this.data.align );
}
} );
}
} );
}}}
The ""with"" needs to be replaced with ""width"" on line 44." Bug closed Normal Documentation & Samples 4.3 fixed