PDA

View Full Version : [FIXED-704][3.x] ListView's onResize needs to return if height is too small



meroy
9 Mar 2010, 10:13 PM
IE 6/7 (not tested with IE 8) errors when resizing the browser window containing a ListView. The error occurs during sizing and the browser window height goes pass the bottom of the ListView column headers.

The code in red below solves the problem. I tried 21 and that wasn't enough. 22 does the trick.

src/widgets/list/ListView.js




// private
onResize : function(w, h){
var bd = this.innerBody.dom;
var hd = this.innerHd.dom;
if(!bd || h < 22){
return;
}
...

meroy
9 Mar 2010, 10:35 PM
For a quick test case, use the list view example.

examples/view/list-view.js

Change the panel.render line to the lines below it.



// panel.render(document.body);

new Ext.Viewport({
renderTo: document.body,
layout: 'fit',
items: panel
});

meroy
9 Mar 2010, 10:43 PM
I have a ListView component living in the mid to bottom section of a west border region. I was testing browser sizing and came across this.

meroy
26 Mar 2010, 7:08 PM
Just tested this and the same issue with the current build. I have a work around in the meantime.

meroy
30 Jun 2010, 6:30 AM
Bump

I'm still having to apply the patch in red from the first post.

evant
1 Jul 2010, 5:43 AM
Fixed in SVN.

meroy
1 Jul 2010, 6:53 AM
Very cool. Have verified that it's working without the patch. Thank you very much.