The code in lp.js should be YUIfied and/or split up.

Bug #490398 reported by Henning Eggers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Unassigned

Bug Description

The Javascript code in the following file look quite old and has no general YUI() wrapper although it has been used for/in some functions. I guess it would be consistent with the rest of LP to fix that.

Also, there is application-specific code in here that should probably go into the specific files for each application.

Revision history for this message
Henning Eggers (henninge) wrote :

The file is lib/canonical/launchpad/javascript/lp/lp.js. It probably needs to be moved, too.

Gary Poster (gary)
Changed in launchpad-foundations:
status: New → Triaged
importance: Undecided → Low
Curtis Hovey (sinzui)
Changed in launchpad:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.