From b56939faf726257f3b2f307bae7e992fbd542ad7 Mon Sep 17 00:00:00 2001 From: scottjehl Date: Fri, 19 Nov 2010 17:31:20 -0500 Subject: [PATCH] updated selectmenu to use position plugin for overlay version --- js/jquery.mobile.forms.select.js | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/js/jquery.mobile.forms.select.js b/js/jquery.mobile.forms.select.js index d9c96dd4..06a51830 100644 --- a/js/jquery.mobile.forms.select.js +++ b/js/jquery.mobile.forms.select.js @@ -196,9 +196,11 @@ $.widget( "mobile.selectmenu", $.mobile.widget, { var self = this, menuHeight = self.list.outerHeight(), + menuWidth = self.list.outerWidth(), scrollTop = $(window).scrollTop(), btnOffset = self.button.offset().top, - screenHeight = window.innerHeight; + screenHeight = window.innerHeight, + screenWidth = window.innerWidth; function focusMenuItem(){ self.list.find( ".ui-btn-active" ).focus(); @@ -221,7 +223,7 @@ $.widget( "mobile.selectmenu", $.mobile.widget, { } else { self.menuType = "overlay"; - + self.screen .height( $(document).height() ) .removeClass('ui-screen-hidden'); @@ -229,11 +231,11 @@ $.widget( "mobile.selectmenu", $.mobile.widget, { self.listbox .append( self.list ) .removeClass( "ui-listbox-hidden" ) - .css({ - top: scrollTop + (screenHeight/2), - "margin-top": -menuHeight/2, - left: window.innerWidth/2, - "margin-left": -1* self.listbox.outerWidth() / 2 + .position({ + my: "center center", + at: "center center", + of: self.button, + collision: "fit" }) .addClass("in");