/*\r
-Copyright (c) 2003-2009, CKSource - Frederico Knabben. All rights reserved.\r
+Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved.\r
For licensing, see LICENSE.html or http://ckeditor.com/license\r
*/\r
CKEDITOR.dialog.add( 'checkbox', function( editor )\r
accessKey : 'V',\r
setup : function( element )\r
{\r
- this.setValue( element.getAttribute( 'value' ) || '' );\r
+ var value = element.getAttribute( 'value' );\r
+ // IE Return 'on' as default attr value.\r
+ this.setValue( CKEDITOR.env.ie && value == 'on' ? '' : value );\r
},\r
commit : function( data )\r
{\r
- var element = data.element;\r
+ var element = data.element,\r
+ value = this.getValue();\r
\r
- if ( this.getValue() )\r
- element.setAttribute( 'value', this.getValue() );\r
+ if ( value && !( CKEDITOR.env.ie && value == 'on' ) )\r
+ element.setAttribute( 'value', value );\r
else\r
- element.removeAttribute( 'value' );\r
+ {\r
+ if ( CKEDITOR.env.ie )\r
+ {\r
+ // Remove attribute 'value' of checkbox #4721.\r
+ var checkbox = new CKEDITOR.dom.element( 'input', element.getDocument() );\r
+ element.copyAttributes( checkbox, { value: 1 } );\r
+ checkbox.replace( element );\r
+ editor.getSelection().selectElement( checkbox );\r
+ data.element = checkbox;\r
+ }\r
+ else\r
+ element.removeAttribute( 'value' );\r
+ }\r
}\r
},\r
{\r
{\r
var replace = CKEDITOR.dom.element.createFromHtml( '<input type="checkbox"'\r
+ ( isChecked ? ' checked="checked"' : '' )\r
- + '></input>', editor.document );\r
+ + '/>', editor.document );\r
+\r
element.copyAttributes( replace, { type : 1, checked : 1 } );\r
replace.replace( element );\r
editor.getSelection().selectElement( replace );\r
}\r
else\r
{\r
- if ( this.getValue() )\r
- element.setAttribute( 'checked', this.getValue() );\r
+ var value = this.getValue();\r
+ if ( value )\r
+ element.setAttribute( 'checked', 'checked' );\r
else\r
element.removeAttribute( 'checked' );\r
}\r