Permalink
Browse files

Menu: Use item.outerHeight() in _scrollIntoView

Fixes #9991
  • Loading branch information...
jzaefferer committed May 6, 2014
1 parent 719150e commit b22280385c05eaf10f4d480c546906b85aa011e1
Showing with 2 additions and 2 deletions.
  1. +1 −1 tests/visual/menu/menu.html
  2. +1 −1 ui/menu.js
@@ -48,7 +48,7 @@
body { font-size:62.5%; }
.ui-menu { width: 200px; margin-bottom: 2em; }
.menu2-container { width: 220px; }
.menu4 { height: 200px; overflow-y: auto; overflow-x: hidden; }
.menu4 { height: 225px; overflow-y: auto; overflow-x: hidden; }
.address-item { border-bottom: 1px solid #999; }
.address-header { display: block; margin-bottom: .2em; font-weight: bold; }
.address-content { display: block; margin-bottom: .2em; padding-left: 10px; }
View
@@ -408,7 +408,7 @@ return $.widget( "ui.menu", {
offset = item.offset().top - this.activeMenu.offset().top - borderTop - paddingTop;
scroll = this.activeMenu.scrollTop();
elementHeight = this.activeMenu.height();
itemHeight = item.height();
itemHeight = item.outerHeight();
if ( offset < 0 ) {
this.activeMenu.scrollTop( scroll + offset );

0 comments on commit b222803

Please sign in to comment.