fixing Issue #53 - html5 <menu> import

swisnl · Jun 7, 2012 · 5636e96 · 5636e96
1 parent 8786bdc
commit 5636e96
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -103,6 +103,10 @@ $.contextMenu is published under the [MIT license](http://www.opensource.org/lic
 
 ## Changelog ##
 
+### 1.5.18 ###
+
+* fixing html5 `<menu>` import (Issue #53)
+
 ### 1.5.17 ###
 
 * fixing `options` to default to `options.trigger = "right"`

diff --git a/src/jquery.contextMenu.js b/src/jquery.contextMenu.js
@@ -1397,7 +1397,7 @@ function menuChildren(items, $children, counter) {
             // http://www.whatwg.org/specs/web-apps/current-work/multipage/interactive-elements.html#the-menu-element
             case 'menu':
                 item = {name: $node.attr('label'), items: {}};
-                menuChildren(item.items, $node.children(), counter);
+                counter = menuChildren(item.items, $node.children(), counter);
                 break;
 
             // http://www.whatwg.org/specs/web-apps/current-work/multipage/commands.html#using-the-a-element-to-define-a-command
@@ -1527,6 +1527,8 @@ function menuChildren(items, $children, counter) {
             items['key' + counter] = item;
         }
     });
+
+    return counter;
 }
 
 // convert html5 menu