Opened 10 years ago

Closed 9 years ago

Last modified 9 years ago

#11062 closed defect (fixed)

FilteringSelect: fires non-necessary onChange event when first load and a click on the dropdown without choose anything

Reported by: richso Owned by: Douglas Hays
Priority: high Milestone: 1.6
Component: Dijit - Form Version: 1.4.3
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

dijit.form.FilteringSelect fires non-necessary onChange event when first load and a click on the dropdown without choosing anything then lost focus(abnormal); after this first fire, a second and after click-and-go sequence will not fire onChange event unless a selection has been made(correct behaviour)

Attachments (1)

filteringselect_faultcase.html (1.3 KB) - added by richso 10 years ago.
fault case

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by bill

Description: modified (diff)

I assume you mean that you open the drop down by clicking on the arrow, but then click a white part of the window to close the dropdown?

I tried http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/_autoComplete.html?testWidget=dijit.form.FilteringSelect and there's no onChange event in that case.

Can you give steps to reproduce against that file, or attach your own test file?

Changed 10 years ago by richso

fault case

comment:2 Changed 10 years ago by richso

supplement info: the attached case has a blank option will trigger the bug, no blank option will not.

comment:3 Changed 9 years ago by bill

Owner: set to Douglas Hays
Summary: dijit.form.FilteringSelect fires non-necessary onChange event when first load and a click on the dropdown without choose anythingFilteringSelect: fires non-necessary onChange event when first load and a click on the dropdown without choose anything

Reproduces for me on 1.5, clicking the drop down arrow and then clicking away to close the drop down w/out selecting anything.

comment:4 Changed 9 years ago by Douglas Hays

Milestone: tbd1.6
Status: newassigned
Version: 1.4.21.4.3

comment:5 Changed 9 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

(In [22550]) Fixes #11062. Initialize _lastReportedValue along with resetValue so that the first onChange is ignored. Added automated testcase.

comment:6 Changed 9 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.