Opened 8 years ago

Closed 8 years ago

Last modified 5 years ago

#13035 closed defect (fixed)

create minimal dojo dependencies for dijit modules

Reported by: Douglas Hays Owned by: Douglas Hays
Priority: high Milestone: 1.7
Component: Dijit Version: 1.6.1
Keywords: Cc:
Blocked By: Blocking:

Description

Most dijit JS files use "dojo" in their define() statement. This should be changed to dojo/_base/kernel and then additional dojo/_base modules added.

Change History (7)

comment:1 Changed 8 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [25106]) Fixes #13035. Within dijit, replaced define(dojo? with "dojo/_base/kernel" et al. !strict

comment:2 Changed 8 years ago by bill

(In [25259]) Fix unneeded require of dojo/fx, and dojo/dnd/move. Refs #13035 !strict.

comment:3 Changed 8 years ago by Rawld Gill

(In [25339]) fixed missing dependency; fixes #13035; !strict

comment:4 Changed 8 years ago by bill

(In [25625]) Remove unneeded require of NodeList. It's not being used directly, and you don't need to require it just to be able to call forEach() or map() on the return value of dojo.query(). Refs #13035 !strict.

comment:5 Changed 8 years ago by bill

(In [25823]) Fixing missing requires for dojo.declare(), refs #13035 !strict. Also removed module return values for deprecated modules, and did various other cleanup.

comment:6 Changed 8 years ago by bill

(In [25858]) fix typo from [25823] (forgot to add declare as function parameter), plus make baseless, refs #13035 !strict.

comment:7 Changed 5 years ago by Bill Keese <bill@…>

In 695bcd4a2126e338864f2889dcda0f436e54eb65/dijit:

Error: Processor CommitTicketReference failed
Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 
Note: See TracTickets for help on using tickets.