Skip to main content

Search and Top Navigation

#7264 closed bug (fixed)

Opened April 17, 2011 08:34PM UTC

Closed April 25, 2011 03:20AM UTC

Last modified June 14, 2011 01:59AM UTC

Spinner returns values beyond min and max, off by one

Reported by: ryanneufeld Owned by:
Priority: minor Milestone: git
Component: [meta] ui.dev Version: git (not yet released)
Keywords: Cc:
Blocked by: Blocking:
Description

See: http://jsfiddle.net/ryanneufeld/m57gP/

When using the spinner if you continue to hit up/down when at the min/max, you get min-1 and max+1 returned in ui.value.

It's expected that when at the min value, min should be returned via the spin callback.

Attachments (0)
Change History (5)

Changed April 25, 2011 03:20AM UTC by scottgonzalez comment:1

resolution: → invalid
status: newclosed

Thanks, but we don't use the bug tracker for plugins that haven't been released yet. I've copied the content of this ticket to the appropriate place: http://wiki.jqueryui.com/w/page/12138077/Spinner#comment1303701545

Changed June 13, 2011 11:49PM UTC by ryanneufeld comment:2

Why is this marked as invalid?

All the places on the jqueryUI site show to report them here... I'm confused.

Changed June 14, 2011 12:43AM UTC by rdworth comment:3

_comment0: See http://blog.jqueryui.com/2010/11/jquery-ui-1-9-milestone-3-spinner/ How To Provide Feedback section1308012271728925

See http://blog.jqueryui.com/2010/11/jquery-ui-1-9-milestone-3-spinner/ How To Provide Feedback section. Of course best would be to test with and provide feedback on 1.9m5 or even later, but that's the post about spinner. The 1.9m5 post has the same instruction on how to provide feedback on testing not yet released components.

Changed June 14, 2011 01:43AM UTC by beatryder comment:4

resolution: invalidfixed

Spinner: modified _spin and _setOption to call new method _trimValue to check for min/max values. Fixed #7264 - Spinner returns values beyond min and max, off by one

Changeset: b0182d78229058fdf61d5f9c3b8a66617d056f30

Changed June 14, 2011 01:59AM UTC by scottgonzalez comment:5

component: ui.core[meta] ui.dev
milestone: 1.9git
version: → git