Opened 12 years ago

Closed 12 years ago

Last modified 9 years ago

#4130 closed defect (fixed)

Spinner: displays NaN

Reported by: haysmark Owned by: Douglas Hays
Priority: high Milestone: 1.0
Component: Dijit - Form Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

In test_Spinner, the Spinner with no attributes displays NaN. This wouldn't be so bad if you could spin out of a NaN. Maybe it should default to 0 so it is usable?

Change History (5)

comment:1 Changed 12 years ago by bill

Hmm, I thought that an NaN value would display as a blank input area. Especially that's how a plain NumberTextbox? should work, in order to support optional number fields.

comment:2 Changed 12 years ago by bill

Luckily this isn't a problem with NumberSpinner?, which displays an empty input field. Technically a spinner could be blank but it's a unusual case so 1.0 is an OK milestone for this.

comment:3 Changed 12 years ago by Douglas Hays

Owner: changed from haysmark to Douglas Hays

comment:4 Changed 12 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [10371]) Fixes #4130. Removed value from the Spinner template. Added filter() to NumberTextboxMixin? to change NaN to . Changed test_Spinner.html to have a form field view action.

comment:5 Changed 9 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.