ScrollSupport for Vertical[Horizontal]LayoutContainer not implemented?

[raivis]

I don't know if it is not implemented or if it is supposed to be like that and one should implement ScrollSupport himself.
ScrollSupport for CssFloatLayoutContainer and FlowLayoutContainer:

@Override
public ScrollSupport getScrollSupport() {
    if (scrollSupport == null) {
        scrollSupport = new DefaultScrollSupport(getContainerTarget());
    }
    return scrollSupport;
}

ScrollSupport for PortalLayoutContainer:

public AutoScrollSupport getAutoScrollSupport() {
    if (scrollSupport == null) {
        scrollSupport = new AutoScrollSupport(getElement());
    }
    return scrollSupport;
}

ScrollSupport for VerticalLayoutContainer and HorizontalLayoutContainer:

@Override
public ScrollSupport getScrollSupport() {
    return scrollSupport;
}

ScrollSupport for last two is never created thus null is returned. Ofcourse one could create his/her own ScrollSupport and set it manually. Just wanted to know if it is supposed to be that way or it is not yet implemented in Vertcal and Horizontal LayoutContainers?

[MarcT]

I am also looking for an answer to this question. Seems like an oversight on Sencha's part, as adding scrolling to VLC & HLC should be as easy as possible (in fact, I'd really like them to just expose the setScrollMode() function directly from the container).

But even if Sencha has decided not to make it easy for us, a definite answer would be helpful.

[nbuesing]

I too would like to know if this is something planned to be implemented, or if this is intentional?

Can someone please given an update on this.

Thanks.

[WesleyMoy]

Thanks for pointing this out. I've filed a ticket against the team to take a look at why this is inconsistent. I'll update this thread with any changes we make.

[lefke123]

We've just encountered this issue as well, and looking at the most recent code (revision 2195) it's not included yet. As a workaround, just set the scroll support with:

container.setScrollSupport(new DefaultScrollSupport(container.getElement()));

But I agree with MarcT that setScrollMode would be an excellent delegate in the container.

I'd like to know if this is something that will be included in the final 3.0 beta, or not? Thanks in advance.

[darrellmeyer]

getScrollSupport now creates a new instance as needed everywhere. We also added get / set ScrollMode delegates to the container's that implement HasScrollSupport. Changes are in SVN and will go out in next release.

[WesleyMoy]

This inconsistency has been corrected in the Ext GWT 3.0 Release Candidate. Please upgrade your copy of Ext GWT and try your test case again. While we're confident that we've addressed this issue, please reply if you notice any continued problems after upgrading. Again, thanks for taking the time to report and describe this bug.