637570 - Disco pane js is less than ideal

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect, P4)

Description

[Jeff Balogh (:jbalogh)]

In addition to the sadness of bug 595991, the disco pane:

1. Has multiple js bundles
2. Loads jquery-ui
3. Loads amo2009/addons.js (to what end?)
4. Loads underscore.js
5. Contains discovery.js and discovery_pane.js (what goes where and why?)

Comment 1

[Wil Clouser [:clouserw]]

cvan:  it sounds like you've already done some work here.  Can you give us an update and eta?

Comment 2

[Christopher Van Wiemeersch [:cvan]]

https://github.com/jbalogh/zamboni/commit/749187

1) I've consolidated the `discovery-pane` and `discovery-addon` bundles into one `discovery` bundle.
2) Removed the jquery-ui include.
3) Removed `amo/2009/addon.js` include.
4) Renamed `discovery_pane.js` to a more appropriate name, `discovery_addons.js`. What was once `discovery.js` is now `discovery_pane.js`.

I've cleaned up and organized the Recommendations code into a nice closure, which I will introduce in a commit next week.

I've also written a Storage object that abstracts the localStorage getItem/setItem/removeItem methods and falls back to cookie storage. I'd like to push this early next week as well, as it'd nicely declutter both JS files.