X-Git-Url: http://git.ithinksw.org/extjs.git/blobdiff_plain/6746dc89c47ed01b165cc1152533605f97eb8e8d..HEAD:/src/layout/container/AbstractContainer.js

diff --git a/src/layout/container/AbstractContainer.js b/src/layout/container/AbstractContainer.js
index 9c72c9b7..7ec5758a 100644
--- a/src/layout/container/AbstractContainer.js
+++ b/src/layout/container/AbstractContainer.js
@@ -16,8 +16,8 @@ If you are unsure which license is appropriate for your use, please contact the
  * @class Ext.layout.container.AbstractContainer
  * @extends Ext.layout.Layout
  * Please refer to sub classes documentation
+ * @private
  */
-
 Ext.define('Ext.layout.container.AbstractContainer', {
 
     /* Begin Definitions */
@@ -44,7 +44,7 @@ Ext.define('Ext.layout.container.AbstractContainer', {
      * @cfg {String} itemCls
      * <p>An optional extra CSS class that will be added to the container. This can be useful for adding
      * customized styles to the container or any of its children using standard CSS rules. See
-     * {@link Ext.Component}.{@link Ext.Component#ctCls ctCls} also.</p>
+     * {@link Ext.Component}.{@link Ext.Component#componentCls componentCls} also.</p>
      * </p>
      */
 
@@ -63,26 +63,33 @@ Ext.define('Ext.layout.container.AbstractContainer', {
     /**
      * <p>Returns an array of child components either for a render phase (Performed in the beforeLayout method of the layout's
      * base class), or the layout phase (onLayout).</p>
-     * @return {Array} of child components
+     * @return {Ext.Component[]} of child components
      */
     getLayoutItems: function() {
         return this.owner && this.owner.items && this.owner.items.items || [];
     },
 
+    /**
+     * Containers should not lay out child components when collapsed.
+     */
+    beforeLayout: function() {
+        return !this.owner.collapsed && this.callParent(arguments);
+    },
+
     afterLayout: function() {
         this.owner.afterLayout(this);
     },
     /**
      * Returns the owner component's resize element.
-     * @return {Ext.core.Element}
+     * @return {Ext.Element}
      */
      getTarget: function() {
          return this.owner.getTargetEl();
      },
     /**
-     * <p>Returns the element into which rendering must take place. Defaults to the owner Container's {@link Ext.AbstractComponent#targetEl}.</p>
+     * <p>Returns the element into which rendering must take place. Defaults to the owner Container's target element.</p>
      * May be overridden in layout managers which implement an inner element.
-     * @return {Ext.core.Element}
+     * @return {Ext.Element}
      */
      getRenderTarget: function() {
          return this.owner.getTargetEl();