Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#15874 closed defect (fixed)

_ScrollableMixin missing dojox dependency

Reported by: haysmark Owned by: Patrick Ruzand
Priority: undecided Milestone: 1.8
Component: DojoX Mobile Version: 1.7.1
Keywords: Cc:
Blocked By: Blocking:

Description

In 1.7.1, _ScrollableMixin uses but does not define "dojox". If dojox is globally named something else due to package mapping, you will get an error on load. I see two possible fixes:

  1. Load ../main, which gets the dojox object.
  2. use getObject to get dojox through the package map, as with scrollable.js.

In 1.8 everything was completely rewritten so this isn't an issue.

Change History (4)

comment:1 Changed 6 years ago by Patrick Ruzand

Owner: changed from Eric Durocher to Patrick Ruzand
Status: newassigned

comment:2 Changed 6 years ago by cjolif

As this seems to fixed in 1.8.x and nobody crying for it in 1.7 maybe we can close this?

comment:3 Changed 6 years ago by haysmark

Resolution: fixed
Status: assignedclosed

We will require those people in 1.7.X who are messing with custom dojox namespaces to upgrade to 1.8.X.

comment:4 Changed 6 years ago by bill

Milestone: tbd1.8
Note: See TracTickets for help on using tickets.