From a0c856467d7cb19442ac24a5e8eda5a504d474d6 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Thu, 21 Oct 2010 22:15:13 -0400 Subject: [PATCH] Autocomplete: Split menu resizing logic into its own method to create a more logical method to proxy for custom sizing. --- ui/jquery.ui.autocomplete.js | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/ui/jquery.ui.autocomplete.js b/ui/jquery.ui.autocomplete.js index e992aadc3..427ad1a5a 100644 --- a/ui/jquery.ui.autocomplete.js +++ b/ui/jquery.ui.autocomplete.js @@ -326,10 +326,8 @@ $.widget( "ui.autocomplete", { _suggest: function( items ) { var ul = this.menu.element - .empty() - .zIndex( this.element.zIndex() + 1 ), - menuWidth, - textWidth; + .empty() + .zIndex( this.element.zIndex() + 1 ); this._renderMenu( ul, items ); // TODO refresh should check if the active item is still in the dom, removing the need for a manual deactivate this.menu.deactivate(); @@ -338,9 +336,15 @@ $.widget( "ui.autocomplete", { of: this.element }, this.options.position )); - menuWidth = ul.width( "" ).outerWidth(); - textWidth = this.element.outerWidth(); - ul.outerWidth( Math.max( menuWidth, textWidth ) ); + this._resizeMenu(); + }, + + _resizeMenu: function() { + var ul = this.menu.element; + ul.outerWidth( Math.max( + ul.width( "" ).outerWidth(), + this.element.outerWidth() + ) ); }, _renderMenu: function( ul, items ) { -- 2.39.5