View Full Version : Ext.grid.RowExpander and autoHeight

28 Apr 2009, 1:31 PM
I'm using the Ext.grid.RowExpander plugin for my grid and I have autoHeight=true on the grid. It works fine in Firefox. In IE the rows expand ok, but when collapsed the vertical space that the expanded row takes up remains even though the row has been hidden. Any clues on how to fix this?

28 Apr 2009, 1:41 PM
Here is an example showing the problem:

Maybe I'm approaching this the wrong way? What I need is for some rows in my grid to be expandable. When expanded they need to show X more rows of data, aligned to the columns. My current approach, shown in the example above, has a few problems:

1) It doesn't collapse correctly in IE.
2) When columns are resized/removed the column widths in the expanded tables are incorrect.
3) When columns are sorted, the expanded tables disappear for some reason.

Does anyone have suggestions on a different approach I could use to meet my needs?

29 Apr 2009, 10:05 AM
Can anyone help or give their 2 cents? :)

17 May 2009, 11:19 AM
As to the collapsing issue:
AFAICT by looking at your table, IE does not collapse the 'td' wrapper of your child table div wrapper.
I think that it has something to do with the 'visibility' value.

You can apply one of the following CSS styles in order to try and fix your problem:

.x-grid3-row-collapsed .x-grid3-row-body

.x-grid3-row x-grid3-row-collapsed .x-grid3-body-cell
display: none;
As per the other issues that you have mentioned, I cannot help at the moment, since I am not familiar with the way in which the nested tables are rendered (they seem to be rendered on expand only, probably not handling the resize/order events).

Probably some tweaking is required for the plugin.
I am also a bit concerned about the way the sub-tables are destroyed. It could lead to memory leaks (I am not 100% sure about this, as I haven't dug deep enough).
I might take a closer look at it in the near future.
If I do and in case I get to solve those issues, I will try not to forget to leave an update here.