Opened 8 years ago

Closed 8 years ago

#13365 closed defect (fixed)

Allow Multiple dojox.mobile.ProgressIndicator

Reported by: Ming Zhe Huang Owned by: ykami
Priority: high Milestone: 1.7
Component: DojoX Mobile Version: 1.7.0b1
Keywords: 1.7-mobile Cc:
Blocked By: Blocking:

Description

There could be more than one progress indicator, when loading different content in devices with relatively larger screen. However, dojox.mobile.ProgressIndicator? is a singleton. It's supposed to use the following code to get the ProgressIndicator?. var prog = dojox.mobile.ProgressIndicator?.getInstance().

I tried to use the following code to get a ProgressIndicator?, which almost works except that the 2nd time you show a progress indicator, it won't scroll. Please see the attached test case for details. var prog = new dojox.mobile.ProgressIndicator?();

Attachments (1)

test_progress-indicator2.html (2.8 KB) - added by Ming Zhe Huang 8 years ago.
Test case for new a ProgressIndicator?

Download all attachments as: .zip

Change History (3)

Changed 8 years ago by Ming Zhe Huang

Test case for new a ProgressIndicator?

comment:1 Changed 8 years ago by ykami

Keywords: 1.7-mobile added
Milestone: tbd1.7
Status: newassigned

It is not an expected usage as you know. But I will (only) save your 2nd indicator, since the fix will be trivial. :-)

btw, the attachment was identical to tests/test_progress-indicator.html.

comment:2 Changed 8 years ago by ykami

Resolution: fixed
Status: assignedclosed

(In [25724]) Fixes #13365 !strict Initialize _bar in the constructor so as not to be shared among instances.

Note: See TracTickets for help on using tickets.