Ticket #6082: 6082_2.patch

_source/core/config.js

@@ -330 +330 @@
         baseFloatZIndex : 10000
 };
 
+/**
+ * Indicates that some of the editor features, like alignement and text
+ * direction, should used the "computed value" of the feature to indicate it's
+ * on/off state, instead of using the "real value".
+ *
+ * If enabled, in a left to right written document, the "Left Justify"
+ * alignment button will show as active, even if the aligment style is not
+ * explicitly applied to the current paragraph in the editor.
+ * @name CKEDITOR.config.useComputedState
+ * @type Boolean
+ * @default true
+ * @example
+ * config.useComputedState = false;
+ */
+
 // PACKAGER_RENAME( CKEDITOR.config )

_source/plugins/bidi/plugin.js

@@ -17 +17 @@
 
         function getState( editor, path, dir )
         {
-                var selection = editor.getSelection(),
-                        ranges = selection.getRanges();
+                var useComputedState = editor.config.useComputedState,
+                        selectedElement;
 
-                var selectedElement = ranges && ranges[ 0 ].getEnclosedNode();
+                useComputedState = useComputedState === undefined || useComputedState;
 
-                // If this is not our element of interest, apply to fully selected elements from guardElements.
-                if ( !selectedElement || selectedElement
-                                && !( selectedElement.type == CKEDITOR.NODE_ELEMENT && selectedElement.getName() in directSelectionGuardElements )
-                        )
-                        selectedElement = getFullySelected( selection, guardElements );
+                if ( useComputedState )
+                {
+                        var selection = editor.getSelection(),
+                                ranges = selection.getRanges();
 
+                        selectedElement = ranges && ranges[ 0 ].getEnclosedNode();
+
+                        // If this is not our element of interest, apply to fully selected elements from guardElements.
+                        if ( !selectedElement || selectedElement
+                                        && !( selectedElement.type == CKEDITOR.NODE_ELEMENT && selectedElement.getName() in directSelectionGuardElements )
+                                )
+                                selectedElement = getFullySelected( selection, guardElements );
+                }
+
                 selectedElement = selectedElement || path.block || path.blockLimit;
 
                 if ( !selectedElement || selectedElement.getName() == 'body' )
                         return CKEDITOR.TRISTATE_OFF;
 
-                return ( selectedElement.getComputedStyle( 'direction' ) == dir ) ?
+                selectedElement = useComputedState ?
+                        selectedElement.getComputedStyle( 'direction' ) :
+                        selectedElement.getStyle( 'direction' ) || selectedElement.getAttribute( 'dir' );
+
+                return ( selectedElement == dir ) ?
                         CKEDITOR.TRISTATE_ON : CKEDITOR.TRISTATE_OFF;
         }
 
         function switchDir( element, dir, editor )
         {
-                var dirBefore = element.getComputedStyle( 'direction' );
+                var dirBefore = element.getComputedStyle( 'direction' ),
+                        currentDir = element.getStyle( 'direction' ) || element.getAttribute( 'dir' ) || '';
 
-                if ( element.hasAttribute( 'dir' ) && element.getAttribute( 'dir' ).toLowerCase() == dir )
+                element.removeStyle( 'direction' );
+
+                if ( currentDir.toLowerCase() == dir )
                         element.removeAttribute( 'dir' );
                 else
                         element.setAttribute( 'dir', dir );

_source/plugins/justify/plugin.js

@@ -13 +13 @@
 
         function getState( editor, path )
         {
-                var firstBlock = path.block || path.blockLimit;
+                var firstBlock = path.block || path.blockLimit,
+                        useComputedState = editor.config.useComputedState,
+                        currentAlign;
 
+                useComputedState = useComputedState === undefined || useComputedState;
+
                 if ( !firstBlock || firstBlock.getName() == 'body' )
                         return CKEDITOR.TRISTATE_OFF;
 
-                var currentAlign = firstBlock.getComputedStyle( 'text-align' ).replace( alignRemoveRegex, '' );
-                if ( ( !currentAlign && isDefaultAlign( this, firstBlock ) ) || currentAlign == this.value )
+                currentAlign = useComputedState ?
+                        firstBlock.getComputedStyle( 'text-align' ) :
+                        firstBlock.getStyle( 'text-align' ) || firstBlock.getAttribute( 'align' ) || '';
+                
+                currentAlign = currentAlign.replace( alignRemoveRegex, '' );
+
+                if ( ( !currentAlign && isDefaultAlign( this, firstBlock, editor ) ) || currentAlign == this.value )
                         return CKEDITOR.TRISTATE_ON;
                 return CKEDITOR.TRISTATE_OFF;
         }
@@ -31 +40 @@
                 command.fire( 'state' );
         }
 
-        function isDefaultAlign( command, element )
+        function isDefaultAlign( command, element, editor )
         {
+                var useComputedState = editor.config.useComputedState;
+
+                if ( useComputedState !== undefined && !useComputedState )
+                        return 0;
+
                 var direction = element.getComputedStyle( 'direction' ),
                         val = command.value;
                 return ( direction == 'rtl' && val == 'right' ) || ( direction == 'ltr' && val == 'left' );
@@ -79 +93 @@
                         var bookmarks = selection.createBookmarks(),
                                 ranges = selection.getRanges( true );
 
-
                         var cssClassName = this.cssClassName,
                                 iterator,
                                 block;
+
                         for ( var i = ranges.length - 1 ; i >= 0 ; i-- )
                         {
                                 iterator = ranges[ i ].createIterator();
@@ -92 +106 @@
                                 {
                                         block.removeAttribute( 'align' );
 
-                                        var isDefault = isDefaultAlign( this, block );
+                                        var isDefault = isDefaultAlign( this, block.getParent(), editor );
 
                                         if ( cssClassName )
                                         {