grid autoHeight

[maulin]

if grid belongs to a parent node with height=0, then setting autoHeight=true on the markup works as expected, EXCEPT that the header row is not visible (going into firebug and changing style on header to have height=33px, makes it show up.) attached is test case.

[maulin]

additionally, if you set autoHeight=number, where number is > than the number of rows returned, you also lose the header (though if number is < rowCount, it works as expected). I also attached a test case for that

[maulin]

autoHeight bug where set to number > rowCount

[maulin]

I think I found the source: Line 587-588 of _Grid show:

// Otherwise, show the header and give it an appropriate height.

this.viewsHeaderNode.style.display = "block";

but the "give it an appropritate height part is never called. I added:

// Otherwise, show the header and give it an appropriate height.

this.viewsHeaderNode.style.display = "block"; this._getHeaderHeight();

and it works fine!(and yes, I have signed the CLA)

[toonetown]

(In [14929]) Fixes #7465 - when changing the display of the header node, recalculate the height. Thanks for the patch !strict