Opened 10 years ago

Closed 10 years ago

#13893 closed defect (fixed)

Checkbox.js is using NodeList.add/removeClass without requiring dojo/NodeList-dom

Reported by: cjolif Owned by: Douglas Hays
Priority: high Milestone: 1.7
Component: Dijit - Form Version: 1.7.0b1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by cjolif)

My application is throwing exception in CheckBox._onFocus/_onBlue because addClass & removeClass methods are not found on the NodeList returned by the query.

This seems to be due to dojo/NodeList-dom not being required by the CheckBox module.

Attached is a simple patch to fix the issue.

Attachments (1)

dijit.CheckBox.patch​ (474 bytes) - added by cjolif 10 years ago.

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by cjolif

Attachment: dijit.CheckBox.patch​ added

comment:1 Changed 10 years ago by cjolif

Description: modified (diff)

comment:2 Changed 10 years ago by Adam Peller

Milestone: tbd → 1.7

comment:3 Changed 10 years ago by Douglas Hays

Resolution: → fixed
Status: new → closed

In [26544]:

Fixes #13893. Add Nodelist-dom module dependency after Nodelist was removed in [25625].

Note: See TracTickets for help on using tickets.