Opened 10 years ago

Closed 9 years ago

Last modified 9 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 9 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 9 years ago by bill

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

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

comment:3 Changed 9 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 9 years ago by bill

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

comment:5 Changed 9 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 9 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.