Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#8280 closed enhancement (wontfix)

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.

Change History (4)

comment:1 Changed 7 years ago by Scott González

Owner: set to tkelley353
Status: newpending

Why are you using Number objects in the first place?

comment:2 in reply to:  1 Changed 7 years ago by tkelley353

Status: pendingnew

Replying to 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.

comment:3 Changed 7 years ago by Scott González

Resolution: wontfix
Status: newclosed

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

comment:4 in reply to:  3 Changed 7 years ago by tkelley353

Replying to 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!

Note: See TracTickets for help on using tickets.