Skip to content
Permalink
Browse files

Menu: Remove the ui-state-active class from the siblings of the newly…

… activated menu item without a delay to fix a bug where mouseover of a menu item directly adjacent to the active item would cause a 1 pixel jump due to both items having a class with a border (ui-state-active and ui-state-focus)
  • Loading branch information...
kborchers committed Jul 13, 2011
1 parent 7856b26 commit bc93b3ff080160e4e8e7a035e2e4421d1521e224
Showing with 2 additions and 0 deletions.
  1. +2 −0 ui/jquery.ui.menu.js
@@ -58,6 +58,8 @@ $.widget( "ui.menu", {
}
var target = $( event.target ).closest( ".ui-menu-item" );
if ( target.length ) {
//Remove ui-state-active class from siblings of the newly focused menu item to avoid a jump caused by adjacent elements both having a class with a border
target.siblings().children( ".ui-state-active" ).removeClass( "ui-state-active" );
self.focus( event, target );
}
})

0 comments on commit bc93b3f

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