Side menu on top and transitions problem

[Carlos Verdier]

Hi

I've been trying the new side menu, and I've noticed what it seems to be a bug. In the following test case, you can see it clearly:

        Form f = new Form(new BorderLayout(BorderLayout.CENTER_BEHAVIOR_CENTER_ABSOLUTE));

        Button btn = new Button("Click me");

        Label lbl = new Label("Replace this");

        Label lbl2 = new Label("Label replaced");        

        Container cont = new Container(new BoxLayout(BoxLayout.Y_AXIS));

        cont.addAll(lbl, btn);

        f.add(BorderLayout.CENTER, cont);

        f.getToolbar().addComponentToSideMenu(new Label("Side Menu"));        

        f.show();

        btn.addActionListener((evt) -> {

            if (cont.contains(lbl))

                cont.replace(lbl, lbl2, CommonTransitions.createSlide(CommonTransitions.SLIDE_HORIZONTAL, false, 300));

            else

                cont.replace(lbl2, lbl, CommonTransitions.createSlide(CommonTransitions.SLIDE_HORIZONTAL, true, 300));                

        });

If you press the button without opening the side menu, the transition works fine. Once you open and close the side menu, the transition no longer works. Reproducible on simulator and devices.

[Shai Almog]

Hi,

there was an issue with form navigation that's fixed in git and should be up this Friday. We might even push it sooner on Thursday as there are a lot of fixes & enhancements.

[Carlos Verdier]

Ok, done.

Thank you

[Carlos Verdier]

Was this issue resolved? I'm still experiencing the same issue.

[Shai Almog]

I think it was. Did you define command behavior in your theme or invoke setCommandBehavior? If so remove it.

[Carlos Verdier]

No, I have no command behavior defined. The test case I posted shows the problem, that persists for me. Could you please give it a try?

[Shai Almog]

That doesn't happen to me anymore in the current code. Does it happen to you with a hello world application after a library update?

[Carlos Verdier]

Yes, it happens after the update. I made a Video working with the code I first posted:

https://www.youtube.com/watch?v=IpngKdVp1OY&feature=youtu.be

[Shai Almog]

Thanks the video helps. I was referring to a different problem.
Can you file an issue on this? I'll try to address it soon.

[Carlos Verdier]

Ok, I've just filed the issue. Thank you.

[Shai Almog]

Thanks it's fixed and should be there by the next update.

[Carlos Verdier]

Hi

After the very last update, this problem seems to be back. Could you please guys take a look to this regression?

Thanks!

[Shai Almog]

Hi,
I just re-ran the test case and it seems to be working fine.

[Carlos Verdier]

That's weird... My libraries are up to date and the issue is there for me.

Are your sure you are not missing something? Have you followed the steps in my video? You need to open the side menu first, and after that animations are broken.

https://www.youtube.com/watch?v=IpngKdVp1OY&feature=youtu.be

[Shai Almog]

Sorry, I totally forgot how to reproduce this and just copied the code. I see it. Was there an issue associated with this?
If not can you please open one?

[Carlos Verdier]

Yes, there was an issue open. I've just added a new comment.

https://github.com/codenameone/CodenameOne/issues/2282

Thanks

[Shai Almog]

It wasn't even closed but it has a reference to the commit which might help tracking down what happened.

[Carlos Verdier]

Ok, thank you

[Carlos Verdier]

Hi again, Shai

I've seen that this issue has been assigned to 4.0. What is the schedule for that version, please? Notice that I can't do any release until this is fixed, as it impacts severely on its behavior.

Thanks

[Shai Almog]

Hi,
this was marked fixed 2 hours before you posted ;-)
In general you can see the milestones planned dates in the github project. 4.0 is the coming milestone which is currently planned for March 7th if I remember correctly but might be bumped a bit because I'm barely scratching my tasks for this release.