SectionStack bug? Calling setItemStartIndent when Stack Section item is a Layout

[beckyo]

Hi guys - thinking this is a bug.

4.1d LGPL 2013-11-07
(Unable to update to a newer build due to this issue: http://forums.smartclient.com/showthread.php?t=28821)

Calling setItemStartIndent(x) on a SectionStack when a SectionStackSection contains an H or V Layout produces a JS NaN error on the layout object's width at the time the stack is added to its parent.

Code sample:

public class TestLayout extends VLayout
{
    public TestLayout()
    {
        Label itemLabel = new Label("Test Item");
        itemLabel.setWidth100();
    
        HLayout testItem = new HLayout();
        testItem.setHeight(20);
        testItem.setWidth100();  //commenting this out makes no difference
        testItem.addMember(itemLabel);
    
        SectionStackSection testSection = new SectionStackSection("Test Section"); 
        testSection.setCanCollapse(true);
        testSection.addItem(testItem);

        SectionStack testStack = new SectionStack();
        testStack.setItemStartIndent(4); 
        testStack.addSection(testSection);

        addMember(testStack);
    }
}

Error/Warning info (isc_HLayout_177 is the JS instance of testItem):

com.smartgwt.client.core.JsObject$SGWT_WARN: 10:00:11.578:WARN:HLayout:isc_HLayout_177:ignoring bad or negative width: NaN
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
at com.google.gwt.dev.shell.MethodAdaptor.invoke(MethodAdaptor.java:105)
at com.google.gwt.dev.shell.MethodDispatch.invoke(MethodDispatch.java:71)
at com.google.gwt.dev.shell.OophmSessionHandler.invoke(OophmSessionHandler.java:172)
at com.google.gwt.dev.shell.BrowserChannelServer.reactToMessagesWhileWaitingForReturn(BrowserChannelServer.java:338)
at com.google.gwt.dev.shell.BrowserChannelServer.invokeJavascript(BrowserChannelServer.java:219)
at com.google.gwt.dev.shell.ModuleSpaceOOPHM.doInvoke(ModuleSpaceOOPHM.java:136)
at com.google.gwt.dev.shell.ModuleSpace.invokeNative(ModuleSpace.java:571)
at com.google.gwt.dev.shell.ModuleSpace.invokeNativeObject(ModuleSpace.java:279)
at com.google.gwt.dev.shell.JavaScriptHost.invokeNativeObject(JavaScriptHost.java:91)
at com.smartgwt.client.widgets.layout.SectionStack.create(SectionStack.java)
at com.smartgwt.client.widgets.BaseWidget.getOrCreateJsObj(BaseWidget.java:441)
at com.smartgwt.client.widgets.layout.Layout.addMember(Layout.java:1299)
...

[Isomorphic]

Hi,
you were right, it was a bug. We have already fixed it, so you will find it solved in the next nightly build release.

Thanks,

[beckyo]

Great, thank you.