Skip to content

Commit faa4b97

Browse files
committed
Menu: Rename nested.html to flyoutmenu.html, updated headers
1 parent f8091f7 commit faa4b97

File tree

4 files changed

+8
-14
lines changed

4 files changed

+8
-14
lines changed

tests/visual/index.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,8 @@ <h2>Widgets</h2>
4646
<a href="menu/menu.html">Menu</a>
4747
<ul>
4848
<li><a href="menu/contextmenu.html">Context Menu</a></li>
49-
<li><a href="menu/nested.html">Nested Menu</a></li>
49+
<li><a href="menu/flyoutmenu.html">Flyout Menu</a></li>
50+
<li><a href="menu/menubar.html">Menubar</a></li>
5051
<li><a href="menu/drilldown.html">Drilldown Menu</a></li>
5152
</ul>
5253
</li>
File renamed without changes.

tests/visual/menu/flyoutmenu.js

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,7 @@
11
/*
2-
* jQuery UI flyout menu
3-
* - written for jQuery UI 1.9 milestone 2 using the widget factory
4-
* Dual licensed under the MIT (MIT-LICENSE.txt)
5-
* and GPL (GPL-LICENSE.txt) licenses.
6-
*
7-
* modified from: http://view.jqueryui.com/menu/tests/visual/menu/nested.html
8-
* by: Michael Lang, http://nexul.com/
2+
* jQuery UI flyoutmenu
93
*
4+
* backported from Michael Lang's fork: http://www.nexul.com/prototypes/toolbar/demo.html
105
*/
116
(function($) {
127

@@ -64,11 +59,13 @@ $.widget("ui.flyoutmenu", {
6459
return value.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, "\\$&");
6560
}
6661
var match = self.activeItem.parent("ul").children("li").filter(function() {
62+
// TODO why filter child anchor here, but not in the filter below?
6763
return new RegExp("^" + escape(character), "i").test($("a", this).text());
6864
});
6965
var match = skip && match.index(self.active.next()) != -1 ? match.next() : match;
7066
if (!match.length) {
7167
character = String.fromCharCode(event.keyCode);
68+
// TODO why use self.widget() here instead of self.activeItem??
7269
match = self.widget().children("li").filter(function() {
7370
return new RegExp("^" + escape(character), "i").test($(this).text());
7471
});

tests/visual/menu/menubar.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,7 @@
11
/*
2-
* jQuery UI flyout menu
3-
* - written for jQuery UI 1.9 milestone 2 using the widget factory
4-
*
5-
* Copyright (c) 2010 Michael Lang, http://nexul.com/
6-
* Dual licensed under the MIT (MIT-LICENSE.txt)
7-
* and GPL (GPL-LICENSE.txt) licenses.
2+
* jQuery UI menubar
83
*
4+
* backported from Michael Lang's fork: http://www.nexul.com/prototypes/toolbar/demo.html
95
*/
106
(function($) {
117

0 commit comments

Comments
 (0)