Skip to main content

Search and Top Navigation

#8280 closed enhancement (wontfix)

Opened April 27, 2012 08:30PM UTC

Closed April 27, 2012 09:13PM UTC

Last modified April 27, 2012 09:19PM UTC

Consider adding support for Number object for value property

Reported by: tkelley353 Owned by: tkelley353
Priority: minor Milestone: 1.9.0
Component: ui.progressbar Version: 1.8.19
Keywords: Cc:
Blocked by: Blocking:
Description

The docs on the web site indicate the type of the "value" option is "Number". However, a quick look at the source shows that it's looking for "number" via typeof. It would be great if some more flexible data type support was added.

Attachments (0)
Change History (4)

Changed April 27, 2012 08:46PM UTC by scottgonzalez comment:1

owner: → tkelley353
status: newpending

Why are you using Number objects in the first place?

Changed April 27, 2012 09:00PM UTC by tkelley353 comment:2

status: pendingnew

Replying to [comment:1 scott.gonzalez]:

Why are you using Number objects in the first place?

Well, I'm not :-) However, a friend of mine was and after finding out why his progress bar was reverting to a value of zero, I thought it reasonable to assume that some people might have a case to use them. I also think it's reasonable to assume that values might be fed into this component from other libraries which utilize Number objects - in which case it would be necessary to intervene and perform a type conversion.

Changed April 27, 2012 09:13PM UTC by scottgonzalez comment:3

resolution: → wontfix
status: newclosed

In general jQuery and jQuery UI do not explicitly handle the object versions of primitives like strings and numbers.

Changed April 27, 2012 09:19PM UTC by tkelley353 comment:4

Replying to [comment:3 scott.gonzalez]:

In general jQuery and jQuery UI do not explicitly handle the object versions of primitives like strings and numbers.

In that case, consistency rules and I agree. Thanks for providing the best js lib on the planet!