Bug: Appearance of FormItems in ToolStrips

[andyx1975]

Hi,

I tried latest SmartGwt 6.0 build from 09.04.2016. The form items do not appear correct in Toolstrip bars (see Screenshots).

Thanks and best regards
Andy



Wrong Appearance of FormItems:



Correct appearance in previous builds (e.g. 05.04.2016, same code):

            FormItem eventItem = new FormItem( "eventItem" );
            eventItem.setTitle( GWebAdmin.lang.filterEventByName() );
            eventItem.setTitleOrientation( TitleOrientation.TOP );
          
            FormItem checkIncludeFest = new FormItem( "checkIncludeFest" );
            checkIncludeFest.setTitle( "Show Fest");
            checkIncludeFest.setType( "boolean" );
            checkIncludeFest.setTitleOrientation( TitleOrientation.TOP );
            checkIncludeFest.setDefaultValue( false );

            ToolStrip toolBarTop = new ToolStrip();
            toolBarTop.setWidth100();

            toolBarTop.addFormItem( eventItem );
            toolBarTop.addFormItem( checkIncludeFest );

            toolBarTop.draw();

[andyx1975]

Ok I found a solution... replacing "FormItem" against CheckboxItem worked.... I do not know why I used FormItems... anyway it works.

CheckboxItem checkIncludeIndicators = new CheckboxItem( "checkIncludeIndicators" );
            checkIncludeIndicators.setTitle( GWebAdmin.lang.includeIndicators() );
            checkIncludeIndicators.setTitleOrientation( TitleOrientation.TOP );
            checkIncludeIndicators.setDefaultValue( false );
​

[Isomorphic]

We've applied a fix to SGWT 6.0p/SC 11.0p and newer to address this, which will be the nightly builds dated 2016-04-15 and beyond.

Note that creating the specific SGWT wrapper type if you know it, as you show in your second post, is actually better than the first approach because with just the FormItem wrapper, you don't have access to any of the SGWT APIs from the Java subclass (CheckboxItem in this case) corresponding to the live JS object.

[andyx1975]

yeah I know, I do not know why I have chosen the first approach.... anyway... thanks!!!