Opened 11 years ago

Closed 11 years ago

#10947 closed enhancement (fixed)

[cla][patch] dijit.typematic allow specifying "minDelay"

Reported by: Jonathan Bond-Caron Owned by: Douglas Hays
Priority: high Milestone: 1.5
Component: Dijit Version: 1.4.0
Keywords: Cc:
Blocked By: Blocking:

Description

The logic in _fireEventAndReload() hardcodes a 10ms minimum for setTimeout().

This change allows specifying the 'minDelay' so that typematic events never fire below that value (i.e. 30ms).

For some widgets, staying between 30-100ms is acceptable and not "too fast" for the user.

Attachments (1)

typematic_minDelay.patch​ (8.3 KB) - added by Jonathan Bond-Caron 11 years ago.
Add 'minDelay' as an option + use case for Spinner

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by Jonathan Bond-Caron

Attachment: typematic_minDelay.patch​ added

Add 'minDelay' as an option + use case for Spinner

comment:1 Changed 11 years ago by Adam Peller

Owner: set to Douglas Hays

comment:2 Changed 11 years ago by Douglas Hays

Milestone: tbd → 1.5

comment:3 Changed 11 years ago by Douglas Hays

Resolution: → fixed
Status: new → closed

(In [21857]) Fixes #10947. Add optional minDelay parameter to typematic functions to keep the browser from going too fast. Spinner will pass along the new minimumTimeout parameter.

Note: See TracTickets for help on using tickets.