#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:
- Load ../main, which gets the dojox object.
- 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
| Owner: | changed from Eric Durocher to Patrick Ruzand |
|---|---|
| Status: | new → assigned |
comment:2 Changed 6 years ago by
comment:3 Changed 6 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
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
| Milestone: | tbd → 1.8 |
|---|
Note: See
TracTickets for help on using
tickets.
As this seems to fixed in 1.8.x and nobody crying for it in 1.7 maybe we can close this?