fix(context menu): add cursor pointer to menu host (#1662)

akveo · Jul 9, 2019 · 337cacf · 337cacf
1 parent a2e20a0
commit 337cacf
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/framework/theme/components/context-menu/_context-menu.component.theme.scss b/src/framework/theme/components/context-menu/_context-menu.component.theme.scss
@@ -6,6 +6,15 @@
 @import '../../styles/core/mixins';
 
 @mixin nb-context-menu-theme() {
+
+  .context-menu-host {
+    /*
+      Fixes click not being bubbled to the body in Safari.
+      https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html
+    */
+    cursor: pointer;
+  }
+
   nb-context-menu {
     background-color: nb-theme(context-menu-background-color);
     border-color: nb-theme(context-menu-border-color);

diff --git a/src/framework/theme/components/context-menu/context-menu.directive.ts b/src/framework/theme/components/context-menu/context-menu.directive.ts
@@ -9,6 +9,7 @@ import {
   ComponentRef,
   Directive,
   ElementRef,
+  HostBinding,
   Input,
   OnChanges,
   OnDestroy,
@@ -105,6 +106,9 @@ import { NbMenuItem, NbMenuService } from '../menu/menu.service';
 })
 export class NbContextMenuDirective implements NbDynamicOverlayController, OnChanges, AfterViewInit, OnDestroy, OnInit {
 
+  @HostBinding('class.context-menu-host')
+  contextMenuHost = true;
+
   /**
    * Position will be calculated relatively host element based on the position.
    * Can be top, right, bottom and left.