Changeset 5400

Timestamp:

05/04/08 08:15:30 (8 months ago)

Author:

paul.bakaus

Message:

ui-slider: use offsetWidth/height in handleSize for now (this makes performance dramatically better), initBoundaries for steps, steps with numeric value works again

Files:

• trunk/ui/ui.slider.js (modified) (3 diffs)

trunk/ui/ui.slider.js

@@ -23 +23 @@
             this.element.bind("setData.slider", function(event, key, value) {
                 self.options[key] = value;
-                if (/min|max/.test(key))
+                if (/min|max|steps/.test(key))
                     self.initBoundaries();
             }).bind("getData.slider", function(event, key){
@@ -106 +106 @@
             //Calculate stepping based on steps
             o.stepping = {
-                x: o.stepping && o.stepping.x || parseInt(o.stepping, 10) || (o.steps && o.steps.x ? o.realMax.x/o.steps.x : 0),
-                y: o.stepping && o.stepping.y || parseInt(o.stepping, 10) || (o.steps && o.steps.y ? o.realMax.y/o.steps.y : 0)
+                x: o.stepping && o.stepping.x || parseInt(o.stepping, 10) || (o.steps ? o.realMax.x/(o.steps.x || parseInt(o.steps, 10) || o.realMax.x) : 0),
+                y: o.stepping && o.stepping.y || parseInt(o.stepping, 10) || (o.steps ? o.realMax.y/(o.steps.y || parseInt(o.steps, 10) || o.realMax.y) : 0)
             };
         },
@@ -189 +189 @@
         handleSize: function(handle,axis) {
             if(!axis) axis = this.options.axis == "vertical" ? 2 : 1;
-            return $(handle != undefined && handle !== null ? this.handle[handle] : this.currentHandle)[axis == 1 ? "outerWidth" : "outerHeight"]();    
+            return $(handle != undefined && handle !== null ? this.handle[handle] : this.currentHandle)[0][axis == 1 ? "offsetWidth" : "offsetHeight"]; 
         },
         click: function(e) {