Ticket #6082: 6082_2.patch

File 6082_2.patch, 5.6 KB (added by Frederico Caldeira Knabben, 9 years ago)
  • _source/core/config.js

     
    330330        baseFloatZIndex : 10000
    331331};
    332332
     333/**
     334 * Indicates that some of the editor features, like alignement and text
     335 * direction, should used the "computed value" of the feature to indicate it's
     336 * on/off state, instead of using the "real value".
     337 *
     338 * If enabled, in a left to right written document, the "Left Justify"
     339 * alignment button will show as active, even if the aligment style is not
     340 * explicitly applied to the current paragraph in the editor.
     341 * @name CKEDITOR.config.useComputedState
     342 * @type Boolean
     343 * @default true
     344 * @example
     345 * config.useComputedState = false;
     346 */
     347
    333348// PACKAGER_RENAME( CKEDITOR.config )
  • _source/plugins/bidi/plugin.js

     
    1717
    1818        function getState( editor, path, dir )
    1919        {
    20                 var selection = editor.getSelection(),
    21                         ranges = selection.getRanges();
     20                var useComputedState = editor.config.useComputedState,
     21                        selectedElement;
    2222
    23                 var selectedElement = ranges && ranges[ 0 ].getEnclosedNode();
     23                useComputedState = useComputedState === undefined || useComputedState;
    2424
    25                 // If this is not our element of interest, apply to fully selected elements from guardElements.
    26                 if ( !selectedElement || selectedElement
    27                                 && !( selectedElement.type == CKEDITOR.NODE_ELEMENT && selectedElement.getName() in directSelectionGuardElements )
    28                         )
    29                         selectedElement = getFullySelected( selection, guardElements );
     25                if ( useComputedState )
     26                {
     27                        var selection = editor.getSelection(),
     28                                ranges = selection.getRanges();
    3029
     30                        selectedElement = ranges && ranges[ 0 ].getEnclosedNode();
     31
     32                        // If this is not our element of interest, apply to fully selected elements from guardElements.
     33                        if ( !selectedElement || selectedElement
     34                                        && !( selectedElement.type == CKEDITOR.NODE_ELEMENT && selectedElement.getName() in directSelectionGuardElements )
     35                                )
     36                                selectedElement = getFullySelected( selection, guardElements );
     37                }
     38
    3139                selectedElement = selectedElement || path.block || path.blockLimit;
    3240
    3341                if ( !selectedElement || selectedElement.getName() == 'body' )
    3442                        return CKEDITOR.TRISTATE_OFF;
    3543
    36                 return ( selectedElement.getComputedStyle( 'direction' ) == dir ) ?
     44                selectedElement = useComputedState ?
     45                        selectedElement.getComputedStyle( 'direction' ) :
     46                        selectedElement.getStyle( 'direction' ) || selectedElement.getAttribute( 'dir' );
     47
     48                return ( selectedElement == dir ) ?
    3749                        CKEDITOR.TRISTATE_ON : CKEDITOR.TRISTATE_OFF;
    3850        }
    3951
    4052        function switchDir( element, dir, editor )
    4153        {
    42                 var dirBefore = element.getComputedStyle( 'direction' );
     54                var dirBefore = element.getComputedStyle( 'direction' ),
     55                        currentDir = element.getStyle( 'direction' ) || element.getAttribute( 'dir' ) || '';
    4356
    44                 if ( element.hasAttribute( 'dir' ) && element.getAttribute( 'dir' ).toLowerCase() == dir )
     57                element.removeStyle( 'direction' );
     58
     59                if ( currentDir.toLowerCase() == dir )
    4560                        element.removeAttribute( 'dir' );
    4661                else
    4762                        element.setAttribute( 'dir', dir );
  • _source/plugins/justify/plugin.js

     
    1313
    1414        function getState( editor, path )
    1515        {
    16                 var firstBlock = path.block || path.blockLimit;
     16                var firstBlock = path.block || path.blockLimit,
     17                        useComputedState = editor.config.useComputedState,
     18                        currentAlign;
    1719
     20                useComputedState = useComputedState === undefined || useComputedState;
     21
    1822                if ( !firstBlock || firstBlock.getName() == 'body' )
    1923                        return CKEDITOR.TRISTATE_OFF;
    2024
    21                 var currentAlign = firstBlock.getComputedStyle( 'text-align' ).replace( alignRemoveRegex, '' );
    22                 if ( ( !currentAlign && isDefaultAlign( this, firstBlock ) ) || currentAlign == this.value )
     25                currentAlign = useComputedState ?
     26                        firstBlock.getComputedStyle( 'text-align' ) :
     27                        firstBlock.getStyle( 'text-align' ) || firstBlock.getAttribute( 'align' ) || '';
     28               
     29                currentAlign = currentAlign.replace( alignRemoveRegex, '' );
     30
     31                if ( ( !currentAlign && isDefaultAlign( this, firstBlock, editor ) ) || currentAlign == this.value )
    2332                        return CKEDITOR.TRISTATE_ON;
    2433                return CKEDITOR.TRISTATE_OFF;
    2534        }
     
    3140                command.fire( 'state' );
    3241        }
    3342
    34         function isDefaultAlign( command, element )
     43        function isDefaultAlign( command, element, editor )
    3544        {
     45                var useComputedState = editor.config.useComputedState;
     46
     47                if ( useComputedState !== undefined && !useComputedState )
     48                        return 0;
     49
    3650                var direction = element.getComputedStyle( 'direction' ),
    3751                        val = command.value;
    3852                return ( direction == 'rtl' && val == 'right' ) || ( direction == 'ltr' && val == 'left' );
     
    7993                        var bookmarks = selection.createBookmarks(),
    8094                                ranges = selection.getRanges( true );
    8195
    82 
    8396                        var cssClassName = this.cssClassName,
    8497                                iterator,
    8598                                block;
     99
    86100                        for ( var i = ranges.length - 1 ; i >= 0 ; i-- )
    87101                        {
    88102                                iterator = ranges[ i ].createIterator();
     
    92106                                {
    93107                                        block.removeAttribute( 'align' );
    94108
    95                                         var isDefault = isDefaultAlign( this, block );
     109                                        var isDefault = isDefaultAlign( this, block.getParent(), editor );
    96110
    97111                                        if ( cssClassName )
    98112                                        {
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy