Skip to content

Commit

Permalink
Menu: Fixes for destroy method completeness
Browse files Browse the repository at this point in the history
  • Loading branch information
jzaefferer committed Jun 18, 2010
1 parent b376fa8 commit 9ac3578
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions ui/jquery.ui.menu.js
Expand Up @@ -22,7 +22,7 @@ $.widget("ui.menu", {
role: "listbox",
"aria-activedescendant": "ui-active-menuitem"
})
.click(function( event ) {
.bind("click.menu", function( event ) {
if (self.options.disabled) {
return false;
}
Expand All @@ -36,7 +36,7 @@ $.widget("ui.menu", {
this.refresh();

if (!this.options.input) {
this.options.input = this.element.attr("tabindex", 0);
this.options.input = this.element.attr("tabIndex", 0);
}
this.options.input.bind("keydown.menu", function(event) {
if (self.options.disabled) {
Expand Down Expand Up @@ -77,7 +77,7 @@ $.widget("ui.menu", {

this.element
.removeClass("ui-menu ui-widget ui-widget-content ui-corner-all")
.removeAttr("tabindex")
.removeAttr("tabIndex")
.removeAttr("role")
.removeAttr("aria-activedescendant");

Expand All @@ -86,7 +86,7 @@ $.widget("ui.menu", {
.removeAttr("role")
.children("a")
.removeClass("ui-corner-all")
.removeAttr("tabindex")
.removeAttr("tabIndex")
.unbind(".menu");
},

Expand All @@ -100,7 +100,7 @@ $.widget("ui.menu", {

items.children("a")
.addClass("ui-corner-all")
.attr("tabindex", -1)
.attr("tabIndex", -1)
// mouseenter doesn't work with event delegation
.bind("mouseenter.menu", function( event ) {
if (self.options.disabled) {
Expand Down

0 comments on commit 9ac3578

Please sign in to comment.