#4072 closed defect (fixed)
Spinner / Combo / SelectBox can break in IE with some select/mouseup operations
| Reported by: | Sam Foster | Owned by: | bill |
|---|---|---|---|
| Priority: | high | Milestone: | 0.9 |
| Component: | Dijit - Form | Version: | 0.9 |
| Keywords: | ie | Cc: | bill, davidb, Douglas Hays |
| Blocked By: | Blocking: |
Description
If you select the value in any of the text input field-based form widgets, and cause the mouseup to fire outside the widget (e.g. select right to left) you get an "Object doesnt support this property or method"
The while loop in _FormWidget's _onMouse method can get the document node (in ie6, 7), which fails (as it doesnt have a getAttribute method). I'm attaching a patch to also check for node.nodeType === 1 (and that's all the patch does).
Attachments (1)
Change History (5)
Changed 14 years ago by
| Attachment: | formwidget_mouseup_sfoster_20070809.patch added |
|---|
comment:1 Changed 14 years ago by
| Owner: | changed from Sam Foster to bill |
|---|
Assigning to bill for his review.
comment:2 Changed 14 years ago by
Hmm, I'm having trouble reproducing this and also I don't understand the relevance of mouseup, since you are changing the code for the mouseover handler.
comment:3 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
(In [10101]) Fixes #4072: Spinner / Combo / SelectBox? can break in IE with some select/mouseup operations. Patch from Sam Foster (CLA on file)
comment:4 Changed 10 years ago by
| Component: | Dijit → Dijit - Form |
|---|
(CLA) (Patch) one line change to dijit/form/_FormWidget.js to head off [document].getAttribute()