From bcee037715691a8ae3d925777686132c73a819af Mon Sep 17 00:00:00 2001
From: Paul Bakaus <paul.bakaus@googlemail.com>
Date: Thu, 29 Jan 2009 15:13:47 +0000
Subject: slider: fixed orientation detection

---
 ui/ui.slider.js | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'ui/ui.slider.js')

diff --git a/ui/ui.slider.js b/ui/ui.slider.js
index 03d7bc6cc..583f51504 100644
--- a/ui/ui.slider.js
+++ b/ui/ui.slider.js
@@ -20,8 +20,7 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
 		var self = this, o = this.options;
 		this._keySliding = false;
 		this._handleIndex = null;
-		this.orientation = o.orientation == 'auto' ? (this.element[0].offsetWidth/this.element[0].offsetHeight > 1 ? 'horizontal' : 'vertical') : o.orientation;
-
+		this._detectOrientation();
 		this._mouseInit();
 
 		this.element
@@ -249,6 +248,10 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
 		return false;
 
 	},
+	
+	_detectOrientation: function() {
+		this.orientation = this.options.orientation == 'auto' ? (this.element[0].offsetWidth/this.element[0].offsetHeight > 1 ? 'horizontal' : 'vertical') : this.options.orientation;
+	},
 
 	_normValueFromMouse: function(position) {
 
@@ -381,11 +384,11 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
 		switch (key) {
 			case 'orientation':
 
-				this.orientation = this.options.orientation == 'auto' ? (this.element[0].offsetWidth/this.element[0].offsetHeight > 1 ? 'horizontal' : 'vertical') : this.options.orientation;
-
+				this._detectOrientation();
+				
 				this.element
 					.removeClass("ui-slider-horizontal ui-slider-vertical")
-					.addClass("ui-slider-" + this._orientation());
+					.addClass("ui-slider-" + this.orientation);
 				this._refreshValue();
 				break;
 			case 'value':
-- 
cgit v1.2.3