Ticket #8280 (closed enhancement: wontfix)

Opened 13 months ago

Last modified 13 months ago

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:
Blocking: Blocked by:

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

comment:1 follow-up: ↓ 2 Changed 13 months ago by scott.gonzalez

  • Owner set to tkelley353
  • Status changed from new to pending

Why are you using Number objects in the first place?

comment:2 in reply to: ↑ 1 Changed 13 months ago by tkelley353

  • Status changed from pending to new

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 follow-up: ↓ 4 Changed 13 months ago by scott.gonzalez

  • Status changed from new to closed
  • Resolution set to wontfix

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 13 months 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.