React-native-navigation: [v2]SideMenu Bug Report(width option problem)

[spikeisryu]

Issue Description

I think there is a bug related to sidemenu width property.
If there is a width configuration at sidemenu option, then it does not hide the sidemenu properly.

Steps to Reproduce / Code Snippets / Screenshots

When I set a specific width at sidemenu option like this,

sideMenu: {
    left: {
        width: 200,
        visible: false,
    },
},

and try to hide the side menu like this,

    Navigation.mergeOptions(this.props.componentId, {
        sideMenu: {
            left: {
                visible: false
            }
        }
    });

The screen shows like this.

[FILL THIS OUT - It will be extremely helpful]

---

Environment

• React Native Navigation version: 2.0.2526
• React Native version: 0.56.0
• Platform(s) (iOS, Android, or both?): iOS
• Device info (Simulator/Device? OS version? Debug/Release?): Simulator/iPhoneX Debug mode

[cvongrim]

I am running into the exact same issue.

• React Native Navigation version: 2.0.2538
• React Native version: 0.56.0
• Platform(s) (iOS, Android, or both?): iOS
• Device info (Simulator/Device? OS version? Debug/Release?): Simulator/iPhone 6 11.4

[shaliko]

I have the same issue:

React Native Navigation version: 2.0.2538
React Native version: 0.56.0
Platform(s) (iOS, Android, or both?): iOS
Device info (Simulator/Device? OS version? Debug/Release?): Simulator/iPhone 6 11.4

[cesarm16]

there is a solution for this?

[SudoPlz]

I'm experiencing the same issue, setting a width does not affect anything.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest Detox and report back. Thank you for your contributions.

[stale[bot]]

The issue has been closed for inactivity.

[chelovekdrakon]

@spikeisryu @cvongrim @shaliko @SudoPlz @cesarm16

Hi guys!
Just stuck with the same issue.
Anybody found a solution for this? :)

P.S. What is interesting... I got it after update from v2.8.0 -> to v2.14.0.
It worked fine before...

[byonghun]

@chelovekdrakon I am running into the same issue with v.2.14.0-snapshot.246 but only when I push a component after toggling the drawer. If I just toggle without pushing a component after, it works fine.

Has anyone found a solution to this problem?

[chelovekdrakon]

@byonghun I still didn't find a solution. Just rolled back to v2.8.0
If you will find one, please let us know :)

[SudoPlz]

@guyca any ideas?

[byonghun]

@chelovekdrakon will do. If I can't figure it out, I'll probably use v2.8.0 as well. Hopefully, @guyca can help us out!

[flikQ]

Anyone found a solution to this? I find it mind boggling that they haven't fixed this 9 months after the OP. Has V2 been discontinued?

[acollazomayer]

Is this fixed? why does this happen

[flikQ]

@guyca is this error going to remain? it's only been 9+ months.

For debug/fix purposes the sidebar is actually under the main screen so it moves with the center screen FYI.

Why was this issue closed?

[AKACC]

@spikeisryu @cvongrim @shaliko @SudoPlz @cesarm16 @chelovekdrakon

I might solve the problem in my case:
instead of using mergeoptions, write options in Navigation.push() method. This is my example:

 Navigation.push('AppStack', {
        component: {
          name: 'Login',
          passProps: {
            text: 'Pushed screen'
          },
          options: {
            sideMenu: {
                left: {
                    visible: false
                  }
            }
          }
        },
   });

it works fine to me. Hope it helps.

[chelovekdrakon]

@AKACC But what if I don't need to push. Just to show/hide sideMenu)

[yogevbd]

Can someone reproduce this in the playground app and push it to a branch? I can't reproduce

[AKACC]

@chelovekdrakon the problem just happened to me when i tried to push to other screen. showing/hiding sidemenu work perfect with mergeoptions alone.

[dorthwein]

Removing the width resolved this in my particular case where I only set the menu on the root tab. Merge options then worked fine on latest version of RRN

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest Detox and report back. Thank you for your contributions.

[stale[bot]]

The issue has been closed for inactivity.

[chelovekdrakon]

Issue is still not solved.
I keep v2.8.0, because if I upgrade to the latest one --> my sideBar is black.
I can not use it