horizon.js is overloaded
Bug #961509 reported by
Gabriel Hurley
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Dashboard (Horizon) |
Fix Released
|
Wishlist
|
Gabriel Hurley |
Bug Description
We created separate js files to make things cleaner, but then everything ended up being added to horizon.js. Time to do another cleanup pass!
tags: | removed: essex-rc-potential |
Changed in horizon: | |
milestone: | none → folsom-1 |
Changed in horizon: | |
milestone: | folsom-1 → folsom-2 |
Changed in horizon: | |
importance: | Low → Wishlist |
Changed in horizon: | |
status: | Fix Committed → Fix Released |
Changed in horizon: | |
milestone: | folsom-2 → 2012.2 |
To post a comment you must log in.
We should using something like: http:// requirejs. org/
We should move it to a module sort of approach where horizon.js creates the base Horizon object/namespace and then individual modules can add on to that on an as-needed, page-by-page (need-by-need) basis.
Any new code or functionality added to the JS would be done in their own (or respective) modules. EG: a module for quote related JS. Horizon.Quota = {}; Which would encapsulate anything related to Quotas and live in its own quota.js file and would be "Required" as it was needed by the individual pages.