Skip to content
Permalink
Browse files

Menu: Replace ui-state-hover with ui-state-focus. More semantic, looks

the same.
  • Loading branch information...
jzaefferer committed Feb 24, 2011
1 parent 3f353cc commit d74fb20a28b16d9eaca39ca6b1c74d5777b505c7
Showing with 3 additions and 3 deletions.
  1. +1 −1 themes/base/jquery.ui.menu.css
  2. +2 −2 ui/jquery.ui.menu.js
@@ -32,7 +32,7 @@
.ui-menu .ui-menu-item a {
font-weight: normal;
}
.ui-menu .ui-menu-item a.ui-state-hover,
.ui-menu .ui-menu-item a.ui-state-focus,
.ui-menu .ui-menu-item a.ui-state-active {
font-weight: normal;
margin: -1px;
@@ -136,7 +136,7 @@ $.widget("ui.menu", {
}
this.active = item.first()
.children( "a" )
.addClass( "ui-state-hover" )
.addClass( "ui-state-focus" )
.attr( "id", function(index, id) {
return (self.itemId = id || self.menuId + "-activedescendant");
})
@@ -153,7 +153,7 @@ $.widget("ui.menu", {
}

var self = this;
this.active.children( "a" ).removeClass( "ui-state-hover" );
this.active.children( "a" ).removeClass( "ui-state-focus" );
// remove only generated id
$( "#" + self.menuId + "-activedescendant" ).removeAttr( "id" );
this.element.removeAttr( "aria-activedescenant" );

0 comments on commit d74fb20

Please sign in to comment.
You can’t perform that action at this time.