Check that the menu should be visible when setting display properties (#289)

negasora · vkbansal · commit 44a6281c0cd3 · 2019-08-12T22:19:41.000+05:30
diff --git a/src/ContextMenu.js b/src/ContextMenu.js
@@ -78,7 +78,7 @@ export default class ContextMenu extends AbstractMenu {
                 });
             });
         } else {
-            if (!this.menu) return;
+            if (!this.menu || !this.state.isVisible) return;
             this.menu.style.opacity = 0;
             this.menu.style.pointerEvents = 'none';
         }

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ export default class ContextMenu extends AbstractMenu {`
`78`	`78`	`});`
`79`	`79`	`});`
`80`	`80`	`} else {`
`81`		`- if (!this.menu) return;`
	`81`	`+ if (!this.menu \|\| !this.state.isVisible) return;`
`82`	`82`	`this.menu.style.opacity = 0;`
`83`	`83`	`this.menu.style.pointerEvents = 'none';`
`84`	`84`	`}`