PDA

View Full Version : [FIXED-1063] Panel height ignored in 'column' layout



extbio
18 Jun 2010, 9:33 AM
Ext version tested:


Ext 3.2.x all buggy, 3.1.0 works.



Browser versions tested against:


FF and Chrome



Description:


A very simple panel with 'column' layout and 2 child panels configed to be of same height (say 500): in Ext 3.2.0 and higher, the 2nd child panel's height config would be ignored.



Test Case:



Ext.onReady(function(){
var v = new Ext.Panel({
layout: 'column',
border: false,
renderTo: 'test',
items: [
{ title:'first', width: 750, height: 500 },
{ title:'2nd', width: 200, height: 500 }
]
});
});
Steps to reproduce the problem:


Run the code



The result that was expected:


Panels 'first' and '2nd' both are 500px tall



The result that occurs instead:


'first' is 500 but '2nd' has body of height 0


I noticed that in the "layout managers" examples in Ext 3.2.0, 3.2.1 and 3.2.2, the rightmost panel was NOT aligned right (in 3.1.0 it was). But http://www.sencha.com/deploy/dev/examples/layout/column.html apparently was aligned right (even though it was using 3.2.1?). Weird.

BTW I found 3.2.x introduced more bugs than even 3.0.0 which was a quite major upgrade. 3.1.0 was much more stable too. These bugs are really holding us back from deploying 3.2.x

BroadridgeSF1
18 Jun 2010, 9:50 AM
I am noticing that last comment as well, but only in Google Chrome. You said:

I noticed that in the "layout managers" examples in Ext 3.2.0, 3.2.1 and 3.2.2, the rightmost panel was NOT aligned right (in 3.1.0 it was). But http://www.sencha.com/deploy/dev/exa...ut/column.html (http://www.sencha.com/deploy/dev/examples/layout/column.html) apparently was aligned right (even though it was using 3.2.1?). Weird.

Same for me: after an upgrade to 3.2 I see this issue.

When I inspect the elements in chrome: the third column does not get the value for bodyStyle (bodyStyle:'padding:5px') applied to the div:

<div class="x-plain-body x-plain-body-noheader" id="ext-gen37" style="width: 350px; ">

The first two columns do (bodyStyle:'padding:5px 0 5px 5px'):


<div class="x-plain-body x-plain-body-noheader" id="ext-gen35" style="padding-top: 5px; padding-right: 0px; padding-bottom: 5px; padding-left: 5px; width: 345px; ">

BroadridgeSF1
18 Jun 2010, 9:53 AM
This is similar to what I posted here (probably the same issue with Panel):

http://www.sencha.com/forum/showthread.php?101934-OPEN-1061-Ext.ux.Portal-style-issue-in-Google-Chrome

extbio
18 Jun 2010, 10:16 AM
@BroadridgeSF1: You're right, the "layout managers" examples problem IS SPECIFIC to Google Chrome. I was using Chrome to check the examples for 3.2.x installations on our server, while using FF to check http://www.sencha.com/deploy/dev/examples/layout/column.html, therefore I thought the issue was fixed on sencha website. Instead it wasn't fixed.

The panel height issue I posted was cross-browser though. It happens in both FF and Chrome, so it's probably a different issue.

BroadridgeSF1
18 Jun 2010, 10:25 AM
Your initial issue can be seen in almost every browser: Safari, FF, Chrome.

IE is the exception though: when I use IEs developer tools, I see the correct height for the 2nd panel, except when I pick: IE8 Standards. It works okay in IE7 Standard and IE7 Quirks.

All the modern browsers are failing. Old IE is working properly. I bet it works fine in IE6 as well! ;-)

extbio
18 Jun 2010, 10:40 AM
Your initial issue can be seen in almost every browser: Safari, FF, Chrome.

IE is the exception though: when I use IEs developer tools, I see the correct height for the 2nd panel, except when I pick: IE8 Standards. It works okay in IE7 Standard and IE7 Quirks.

All the modern browsers are failing. Old IE is working properly. I bet it works fine in IE6 as well! ;-)

No kidding! Indeed NONE of the issues mentioned above exists in IE6. I didn't even bother to check IE 'cause it has so many CSS issues anyhow, guess double negatives makes a positive!

BTW, now that you started it, I tested it in Opera too and the height worked in Opera too! So it doesn't work in FF and Chrome but does work in IE and Opera. I tested Safari 4.0.3 and it works just like IE and Opera (unlike your result with Safari?)

BroadridgeSF1
18 Jun 2010, 10:43 AM
Ah yes, I just downloaded Safari 5. That is probably why I do see the issue in my Safari. Your "old" Safari is like IE6... :D

Jamie Avins
21 Jun 2010, 9:46 AM
[type]: revert
[module]: Panel
[id]: refs #1063
[desc]: Revert getFrameHeight calculation change.