Skip to content

Commit b3bc487

Browse files
committed
Real fix!
1 parent 298078a commit b3bc487

3 files changed

Lines changed: 9 additions & 3 deletions

File tree

dist/ng-context-menu.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ angular
77
.module('ng-context-menu', [])
88
.factory('ContextMenuService', function() {
99
return {
10+
element: null,
1011
menuElement: null
1112
};
1213
})
@@ -51,6 +52,8 @@ angular
5152
close(ContextMenuService.menuElement);
5253
}
5354
ContextMenuService.menuElement = angular.element(document.getElementById($attrs.target));
55+
ContextMenuService.element = event.target;
56+
console.log('set', ContextMenuService.element);
5457

5558
event.preventDefault();
5659
event.stopPropagation();
@@ -73,7 +76,7 @@ angular
7376
function handleClickEvent(event) {
7477
if (!$scope.disabled() &&
7578
opened &&
76-
(event.button !== 2 || event.target.parent !== ContextMenuService.menuElement[0])) {
79+
(event.button !== 2 || event.target !== ContextMenuService.element)) {
7780
$scope.$apply(function() {
7881
close(ContextMenuService.menuElement);
7982
});

dist/ng-context-menu.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/ng-context-menu.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ angular
77
.module('ng-context-menu', [])
88
.factory('ContextMenuService', function() {
99
return {
10+
element: null,
1011
menuElement: null
1112
};
1213
})
@@ -51,6 +52,8 @@ angular
5152
close(ContextMenuService.menuElement);
5253
}
5354
ContextMenuService.menuElement = angular.element(document.getElementById($attrs.target));
55+
ContextMenuService.element = event.target;
56+
console.log('set', ContextMenuService.element);
5457

5558
event.preventDefault();
5659
event.stopPropagation();
@@ -73,7 +76,7 @@ angular
7376
function handleClickEvent(event) {
7477
if (!$scope.disabled() &&
7578
opened &&
76-
(event.button !== 2 || event.target.parent !== ContextMenuService.menuElement[0])) {
79+
(event.button !== 2 || event.target !== ContextMenuService.element)) {
7780
$scope.$apply(function() {
7881
close(ContextMenuService.menuElement);
7982
});

0 commit comments

Comments
 (0)