ubports/ubuntu-ui-toolkit

AdaptivePageLayout.addPageToNextColumn and addPageToCurrentColumn doesn't return the object incubator

Closed this issue · 3 comments

It seem that the docuimentaion suggest that calling addPageToNextColumn/addPageToCurrentColumn will return the object incubator which then can be use to remove the loaded page however it currently will always return null.

It seems the the issue is that in the code it return wrapper.incubator however this object is never created in the createWrapper function :

        function createWrapper(page, properties) {
            var wrapperObject = pageWrapperComponent.createObject(hiddenPages, {synchronous: !layout.asynchronous});
           ...
           return wrapperObject;

this means that the addPageToNextColumn/addPageToCurrentColumn should actullay return wrapper instead of wrapper.incubator

just to test, does adding asynchronous: false to the AdpativeLayout fix the issue ?

@lduboeuf (as far as i remember) its that it returns the wrong object and that an object that is never created as far as i understood. however if adding asynchronous: false will create the wrapper.incubator` field in the wrapper then maybe we need to add a check to return the incobutor on async=true mode and the wrapper on async=false

Ubuntu UI Toolkit has moved to https://gitlab.com/ubports/core/lomiri-ui-toolkit. If this issue is still relevant, please file it again at https://gitlab.com/ubports/core/lomiri-ui-toolkit/-/issues.