Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-5861 in a recent build.
  1. #11
    Sencha User ykey's Avatar
    Join Date
    Mar 2010
    Location
    USA
    Posts
    245
    Vote Rating
    27
    ykey has a spectacular aura about ykey has a spectacular aura about

      0  

    Default


    Using an anchor layout is not going to work for this because I am dealing with a card layout.

    Here is a better example of what I am trying to achieve:

    Code:
    Ext.onReady(function() {
    	Ext.create('Ext.container.Viewport', {	
    		layout: {
    			type: 'vbox',
    			align: 'stretch'
    		},
    		items: [ 
    			{
    				xtype: 'component',
    				html: 'Header',
    				height: 50
    			},
    			{
    				xtype: 'toolbar',
    				items: [
    					{                                      
    						xtype: 'button',
    						text: 'Screen',
    						handler: function() {
    						   Ext.ComponentQuery.query('#screens')[0].getLayout().setActiveItem('screen1');
    						}
    						
    					}, 
    					{
    						xtype: 'button',
    						text: 'Screen2',
    						handler: function() {
    							Ext.ComponentQuery.query('#screens')[0].getLayout().setActiveItem('screen2');
    						}
    					}
    				]
    			},
    			{
    				xtype: 'container',
    				itemId: 'screens',
    				layout: 'card',			           
    				autoScroll: true,
    				flex: 1,
    				items: [			                
    					{
    						xtype: 'container',	
    						itemId: 'screen1',
    						minHeight: 400,
    						layout: {
    							type: 'vbox',
    							align: 'stretch'
    						},
    						items: [
    							{
    								xtype: 'component',
    								flex: 1,
    								html: 'Screen 1 Part 1',
    								style: 'background-color: orange'
    							},
    							{
    								xtype: 'component',
    								flex: 1,
    								html: 'Screen 1 Part 2',
    								style: 'background-color: green'
    							}
    						]
    					},
    					{
    						xtype: 'container', 
    						itemId: 'screen2',
    						minHeight: 500,
    						layout: {
    							type: 'vbox',
    							align: 'stretch'
    						},
    						items: [
    							{
    								xtype: 'component',
    								height: 425,
    								html: 'Screen 2 Part 1',
    								style: 'background-color: blue'
    							},
    							{
    								xtype: 'component',                                        
    								flex: 1,
    								html: 'Screen 2 Part 2',
    								style: 'background-color: yellow'
    							}
    						]
    					}			                
    				]			        
    			},
    			{
    				xtype: 'component',
    				html: 'Footer',
    				height: 20
    			}
    		]
    	});
    });

  2. #12
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,496
    Vote Rating
    44
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      0  

    Default


    Thanks for that testcase, it will make a good test for the fix.

  3. #13
    Sencha - Ext JS Dev Team dongryphon's Avatar
    Join Date
    Jul 2009
    Posts
    1,295
    Vote Rating
    121
    dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all

      0  

    Default


    You are right that this was a fit layout bug - it never considered that a minWidth/Height could cause overflow and that it should account for that. It needed to account for the presence of scrollbars as it applied the container size to the child as well as when it shrinkwraps the child. Basically, almost a rewrite of the guts of that (formerly simple) layout.

    Great catch!

    Sadly, it won't make it in 4.1.0, but should be in soon after. I have the fixes on a work branch targeting 4.1.1, so it should appear in nightly builds after GA.
    Don Griffin
    Ext JS Development Team Lead

    Check the docs. Learn how to (properly) report a framework issue and a Sencha Cmd issue

    "Use the source, Luke!"

  4. #14
    Sencha - Ext JS Dev Team dongryphon's Avatar
    Join Date
    Jul 2009
    Posts
    1,295
    Vote Rating
    121
    dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all

      0  

    Default


    I should mention that Chrome (only so far) does not want to remove the scrollbar once it goes back to not being needed. I will see what I can do about that, but the only way I've found to remove it is to cycle overflow styles. :P
    Don Griffin
    Ext JS Development Team Lead

    Check the docs. Learn how to (properly) report a framework issue and a Sencha Cmd issue

    "Use the source, Luke!"

  5. #15
    Sencha User ykey's Avatar
    Join Date
    Mar 2010
    Location
    USA
    Posts
    245
    Vote Rating
    27
    ykey has a spectacular aura about ykey has a spectacular aura about

      0  

    Default


    Thanks for the update Don. This is a pretty annoying bug especially during development since bringing up the admin console causes the overflow.

    Just so we are clear this is a regression from behavior in 4.0.7. I am assuming that Sencha will be going back to the dot releases only being available to support subscribers model. That makes me sad. Hopefully the pace for 4.2 will be accelerated compared to 4.1 once it gets out the door otherwise I will not see this fix until April 2013.

  6. #16
    Sencha - Ext JS Dev Team dongryphon's Avatar
    Join Date
    Jul 2009
    Posts
    1,295
    Vote Rating
    121
    dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all dongryphon is a name known to all

      0  

    Default


    The plan is for 4.1.1 to be released to all users.
    Don Griffin
    Ext JS Development Team Lead

    Check the docs. Learn how to (properly) report a framework issue and a Sencha Cmd issue

    "Use the source, Luke!"

  7. #17
    Sencha User ykey's Avatar
    Join Date
    Mar 2010
    Location
    USA
    Posts
    245
    Vote Rating
    27
    ykey has a spectacular aura about ykey has a spectacular aura about

      0  

    Default


    This looks better in 4.1.1 but like Don said the vertical scroll bar is not going away in Chrome. In Firefox and IE everything appears to work fine.

  8. #18
    Sencha User ykey's Avatar
    Join Date
    Mar 2010
    Location
    USA
    Posts
    245
    Vote Rating
    27
    ykey has a spectacular aura about ykey has a spectacular aura about

      0  

    Default


    This is actually worse in Chrome because the scroll bar is now covering up content.

Thread Participants: 3

Tags for this Thread