Grid does not resize with parent widgets

[BryanForbes]

When the grid is parented by a widget that inherits from dijit.layout._LayoutContainer, it does not resize when the parent widget resizes. Rather, you are required to use this code to get it to resize:

dojo.connect(container, "resize", grid, function() {
    grid.resize();
});

This makes using the grid as a standard widget inside container widgets a pain.

[alex]

this bug appears to be somewhat inaccurate. See http://dojotoolkit.org/~alex/anon_view/dojox/grid/tests/test_grid_layout.html

If not parented by a ContentPane? (a known resize() event black-hole), resize events are correctly cascaded.

Marking "invalid".

[BryanForbes]

Reopening and adding a test. This test is exactly like test_grid_layout.html except the bottom container is a dijit.layout.LayoutContainer?. If you move the split up, you'll notice that the bottom grid does not resize.

[bill]

Bryan, this looks wrong:

<div id="bottomRight" dojoType="dijit.layout.LayoutContainer" sizeMin="20" sizeShare="30">
             <div id="grid3" dojoType="dojox.Grid" model="model" structure="layout2"></div>
</div>

You can probably just get rid of the LayoutContainer? and make the Grid a child of the LayoutContainer?'s parent.

[bill]

Marking invalid; feel free to reopen with a valid testcase, as per my advice above.

[guest]

Replying to bill:

Marking invalid; feel free to reopen with a valid testcase, as per my advice above.

Here is a test case (test_grid_layout_3.html) that shows invalid sizing of grid when we try to set it as layoutAlign="client". Grid seems to ignore arguments passed from _LayoutWidget and takes the height from parent node (it should take parent.h-topNode.h-bottom.h).

[guest]

Test case for invalid sizing of grid

[bill]

Ah, I see the problem. resize() should accept an optional argument specfying the size. From _LayoutWidget.js:

resize: function(args){
	// summary:
	//		Explicitly set this widget's size (in pixels),
	//		and then call layout() to resize contents (and maybe adjust child widgets)
	//	
	// args: Object?
	//		{w: int, h: int, l: int, t: int}

But grid doesn't support that.

[guest]

Are there any plans how it would be done? It would be nice if grid will fit whole available client area. Maybe there is a chance to add support for columns that will expand their width if there is more horizontal space available?

[sorvell]

(In [12794]) !strict fixes #4900. Added test in bug to grid tests. Also corrected problem with autoHeight introduced by having grid hidden in an unsized container. Fixed: grid hides in an unsized container on IE.

[sorvell]

[12876] Grid should now size correctly (at least in simple layouts) inside a LayoutContainer? and a BorderContainer?