Bug Tracker

Ticket #2508 (closed bug: invalid)

Opened 9 months ago

Last modified 9 months ago

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

Reported by: joern Assigned to: paul
Type: bug Priority: major
Milestone: Component: ui
Version: Keywords:
Cc: Needs: Review

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

Change History

Changed 9 months ago by joern

  • status changed from new to closed
  • resolution set to invalid

Missed the "o.axis ".

Note: See TracTickets for help on using tickets.