Changeset 7176 for CKEditor/trunk


Ignore:
Timestamp:
08/04/11 06:33:39 (3 years ago)
Author:
garry.yao
Message:

#8124: style fields are now validated.

Location:
CKEditor/trunk
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r7175 r7176  
    4949                <li><a href="http://dev.ckeditor.com/ticket/8155">#8155</a> : Tooltips in the Special Character dialog window corrected.</li> 
    5050                <li><a href="http://dev.ckeditor.com/ticket/8163">#8163</a> : The name of the <code><a href="http://docs.cksource.com/ckeditor_api/symbols/CKEDITOR.config.html#.filebrowserWindowFeatures">filebrowserWindowFeatures</a></code> configuration setting corrected to match the documented name.</li> 
     51                <li><a href="http://dev.ckeditor.com/ticket/8124">#8124</a> : The "style" field in advanced dialog tab are now validated according to CSS style attribute syntax.</li> 
    5152                <li>Updated the following language files:<ul> 
    5253                        <li><a href="http://dev.ckeditor.com/ticket/8128">#8128</a> : Italian;</li> 
  • CKEditor/trunk/_source/lang/en.js

    r7025 r7176  
    124124                invalidCssLength        : 'Value specified for the "%1" field must be a positive number with or without a valid CSS measurement unit (px, %, in, cm, mm, em, ex, pt, or pc).', 
    125125                invalidHtmlLength       : 'Value specified for the "%1" field must be a positive number with or without a valid HTML measurement unit (px or %).', 
     126                invalidInlineStyle      : 'Value specified for the inline style must be one or multiple tuples with the form "name : value" which are separated by semi-colon.', 
    126127 
    127128                // Put the voice-only part of the label in the span. 
  • CKEditor/trunk/_source/plugins/dialog/plugin.js

    r7065 r7176  
    29402940                        numberRegex = /^\d*(?:\.\d+)?$/, 
    29412941                        htmlLengthRegex = /^(((\d*(\.\d+))|(\d*))(px|\%)?)?$/, 
    2942                         cssLengthRegex = /^(((\d*(\.\d+))|(\d*))(px|em|ex|in|cm|mm|pt|pc|\%)?)?$/i; 
     2942                        cssLengthRegex = /^(((\d*(\.\d+))|(\d*))(px|em|ex|in|cm|mm|pt|pc|\%)?)?$/i, 
     2943                        inlineStyleRegex = /^(\s*[\w-]+\s*:\s*[^:;]+(?:;|$))*$/; 
    29432944 
    29442945                CKEDITOR.VALIDATE_OR = 1; 
     
    30303031                        { 
    30313032                                return this.functions( function( val ){ return htmlLengthRegex.test( CKEDITOR.tools.trim( val ) ); }, msg ); 
     3033                        }, 
     3034 
     3035                        'inlineStyle' : function( msg ) 
     3036                        { 
     3037                                return this.functions( function( val ){ return inlineStyleRegex.test( CKEDITOR.tools.trim( val ) ); }, msg ); 
    30323038                        }, 
    30333039 
  • CKEditor/trunk/_source/plugins/dialogadvtab/plugin.js

    r6979 r7176  
    133133                                                'default' : '', 
    134134 
     135                                                validate : CKEDITOR.dialog.validate.inlineStyle( lang.invalidInlineStyle ), 
    135136                                                onChange : function(){}, 
    136137 
  • CKEditor/trunk/_source/plugins/flash/dialogs/flash.js

    r7050 r7176  
    662662                                                        type : 'text', 
    663663                                                        id : 'style', 
     664                                                        validate : CKEDITOR.dialog.validate.inlineStyle( editor.lang.common.invalidInlineStyle ), 
    664665                                                        label : editor.lang.common.cssStyle, 
    665666                                                        setup : loadValue, 
  • CKEditor/trunk/_source/plugins/image/dialogs/image.js

    r6975 r7176  
    13501350                                                        id : 'txtdlgGenStyle', 
    13511351                                                        label : editor.lang.common.cssStyle, 
     1352                                                        validate : CKEDITOR.dialog.validate.inlineStyle( editor.lang.common.invalidInlineStyle ), 
    13521353                                                        'default' : '', 
    13531354                                                        setup : function( type, element ) 
  • CKEditor/trunk/_source/plugins/link/dialogs/link.js

    r7039 r7176  
    11601160                                                                                'default' : '', 
    11611161                                                                                id : 'advStyles', 
     1162                                                                                validate : CKEDITOR.dialog.validate.inlineStyle( editor.lang.common.invalidInlineStyle ), 
    11621163                                                                                setup : setupAdvParams, 
    11631164                                                                                commit : commitAdvParams 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy