Bug Tracker

Opened 14 years ago

Closed 14 years ago

#2591 closed bug (fixed)

Slider size method only returns the width of the slider

Reported by: natecavanaugh Owned by: joern
Priority: major Milestone: 1.2.4
Component: ui Version: 1.2.3
Keywords: Cc:
Blocked by: Blocking:


When checking for a sliders size, the size method only returns it's outerWidth, however, when using a vertical slider, you need the height of the slider, not its width.

The fix is to change this on line 192 in ui.slider.js:

return this.actualSize
(this.actualSize = this.element.outerWidth());

to this:

return this.actualSize
(this.actualSize = (this.options.axis == 'horizontal') ? this.element.outerWidth() : this.element.outerHeight());

Change History (3)

comment:1 Changed 14 years ago by natecavanaug

Somehow the pipes are missing:

return this.actualSize | | (this.actualSize = (this.options.axis == 'horizontal') ? this.element.outerWidth() : this.element.outerHeight());

comment:2 Changed 14 years ago by paul

Owner: changed from paul to joern

comment:3 Changed 14 years ago by joern

Resolution: fixed
Status: newclosed

That got fixed some time ago, see latest revision.

Note: See TracTickets for help on using tickets.