Opened 8 years ago

Closed 8 years ago

#13461 closed defect (wontfix)

Repeated start() of ProgressIndicator can speed up icon spinning

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

Description

Please see the test case. Steps to reproduce:

  1. Click "Start" button twice or more.
  2. You'll find that the progress indicator is spinning much faster.

We should add defensive in ProgressIndicator? to ignore extra start() invocation if the progress indicator is already started.

Attachments (1)

test_progress-indicator2.html​ (1.3 KB) - added by Ming Zhe Huang 8 years ago.

Download all attachments as: .zip

Change History (2)

Changed 8 years ago by Ming Zhe Huang

Attachment: test_progress-indicator2.html​ added

comment:1 Changed 8 years ago by ykami

Resolution: → wontfix
Status: new → closed

That's an interesting behavior.. but please just do not start a progress indicator twice or more. (Or check ProgressIndicator.timer by yourself) Adding some defensive code sounds like a good idea, but for all the applications that have no such programming errors, it's nothing but code bloat.

Note: See TracTickets for help on using tickets.