Opened 12 years ago

Closed 10 years ago

Last modified 10 years ago

#8670 closed enhancement (fixed)

ProgressBar: support set() to update attributes

Reported by: bill Owned by: bill
Priority: high Milestone: 1.6
Component: Dijit Version: 1.3.0b1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

Currently ProgressBar attributes need to be updated through update() method, rather than through the standard attr() method.

Change History (6)

comment:1 Changed 11 years ago by bill

Description: modified (diff)
Milestone: 1.5future
Summary: ProgressBar: attr() doesn't workProgressBar: support attr() to update attributes
Type: defectenhancement

comment:2 Changed 10 years ago by bill

Milestone: future1.6
Owner: set to bill
Status: newassigned

Mostly fixed already in [22522], [22523].

comment:3 Changed 10 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [22549]) Finish cleanup so that set() can be used to adjust all attributes of the ProgressBar, and update() is documented as private (but still supported for backwards compatibility). Fixes #8670 !strict.

comment:4 Changed 10 years ago by bill

Summary: ProgressBar: support attr() to update attributesProgressBar: support set() to update attributes

comment:5 Changed 10 years ago by bill

(In [22667]) Default value of "value" needs to be "" not 0, because if it's 0 it overrides a explicit intermediate:true parameter setting. Refs #8670 !strict.

comment:6 Changed 10 years ago by bill

(In [22668]) Use get('value') rather than this.value in custom label function since this.value isn't getting updated to the latest value. Fallout from trying to deprecate "progress" and "indeterminate" in favor of "value". Refs #8670 !strict.

Note: See TracTickets for help on using tickets.