Skip to main content

Search and Top Navigation

#2508 closed bug (notabug)

Opened March 14, 2008 10:24AM UTC

Closed March 14, 2008 11:08AM UTC

ui.slider - Specified axis gets overwritten by "auto detection"

Reported by: joern Owned by: paul
Priority: major Milestone:
Component: ui.core Version:
Keywords: Cc:
Blocked by: Blocking:
Description

This code initializes the options:

//Prepare the passed options
this.options = $.extend({}, options);
var o = this.options;
$.extend(o, {
	axis: o.axis || (element.offsetWidth < element.offsetHeight ? 'vertical' : 'horizontal'),
	maxValue: !isNaN(parseInt(o.maxValue,10)) ? parseInt(o.maxValue,10) :  100,
	minValue: parseInt(o.minValue,10) || 0
});

The docs say, for axis:

Normally you don't need to set this option because the plugin detects the slider orientation automatically. If the orientation is not correctly detected you can set this option to 'horizontal' or 'vertical'.

Looks to me like the specified option is always overwritten by the autodetection.

Attachments (0)
Change History (1)

Changed March 14, 2008 11:08AM UTC by joern comment:1

resolution: → invalid
status: newclosed

Missed the "o.axis ||".