Skip to content

Commit 1ac4e60

Browse files
committed
Fix context menu extension's focus problem
Close #1018
1 parent 0d84282 commit 1ac4e60

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

3rd-party/extensions/contextmenu/contextmenu.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@
1212

1313
<!-- jquery-contextmenu (https://github.com/swisnl/jQuery-contextMenu) -->
1414
<link rel="stylesheet"
15-
href="//cdn.jsdelivr.net/npm/jquery-contextmenu@2.6.4/dist/jquery.contextMenu.min.css" />
16-
<script src="//cdn.jsdelivr.net/npm/jquery-contextmenu@2.6.4/dist/jquery.contextMenu.min.js">
15+
href="//cdn.jsdelivr.net/npm/jquery-contextmenu@2.9.2/dist/jquery.contextMenu.min.css" />
16+
<script src="//cdn.jsdelivr.net/npm/jquery-contextmenu@2.9.2/dist/jquery.contextMenu.min.js">
1717
</script>
1818

1919
<script src="js/jquery.fancytree.contextMenu.js"></script>

3rd-party/extensions/contextmenu/js/jquery.fancytree.contextMenu.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
if (node) {
1919
$.contextMenu("destroy", "." + selector);
2020

21-
node.setFocus(true);
21+
// node.setFocus(true);
2222
node.setActive(true);
2323

2424
$.contextMenu({

CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,8 @@
55
* [Fixed] ext-dnd5: dropEffectCallback=none was not reset in some cases
66
* [Changed] #1005 Cast key to string in getNodeByKey()
77
* [Added] #1012 `dnd5.dropMarkerParent` allows usage in Webcomponents (i.e. shadow DOM)
8-
* [Added] #1017 `copyFunctionsToData` allows also copying functions to the data property of the node
8+
* [Added] #1017 `copyFunctionsToData` allows also copying functions to the data property of the node
9+
* [Fixed] #1018 ContextMenu extension always focuses the first node in the tree
910

1011
# 2.35.0 / 2020-03-27
1112
* [Changed] The `enableAspx` option will default to 'false' in the future.

0 commit comments

Comments
 (0)