Merge lp:~jcsackett/launchpad/kill-lazr-1 into lp:launchpad
| Status: | Merged |
|---|---|
| Approved by: | Curtis Hovey on 2012-08-23 |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 15880 |
| Proposed branch: | lp:~jcsackett/launchpad/kill-lazr-1 |
| Merge into: | lp:launchpad |
| Diff against target: |
902 lines (+94/-99) 46 files modified
buildout-templates/bin/combine-css.in (+1/-1) lib/lp/answers/javascript/tests/test_answercontacts.html (+1/-1) lib/lp/answers/javascript/tests/test_subscribers.html (+1/-1) lib/lp/app/javascript/activator/activator.js (+2/-2) lib/lp/app/javascript/activator/assets/skins/sam/activator-skin.css (+1/-1) lib/lp/app/javascript/activator/tests/test_activator.html (+1/-1) lib/lp/app/javascript/anim/tests/test_anim.html (+1/-1) lib/lp/app/javascript/choiceedit/choiceedit.js (+2/-3) lib/lp/app/javascript/choiceedit/tests/test_choiceedit.html (+1/-1) lib/lp/app/javascript/confirmationoverlay/tests/test_confirmationoverlay.html (+1/-1) lib/lp/app/javascript/formoverlay/tests/test_formoverlay.html (+1/-1) lib/lp/app/javascript/formwidgets/tests/test_formwidgets.html (+1/-1) lib/lp/app/javascript/inlineedit/assets/skins/sam/editor-skin.css (+2/-2) lib/lp/app/javascript/inlineedit/editor.js (+5/-5) lib/lp/app/javascript/inlineedit/tests/test_inline_edit.html (+1/-1) lib/lp/app/javascript/picker/assets/skins/sam/picker-skin.css (+1/-1) lib/lp/app/javascript/picker/picker.js (+9/-8) lib/lp/app/javascript/picker/tests/test_personpicker.html (+1/-1) lib/lp/app/javascript/picker/tests/test_picker.html (+1/-1) lib/lp/app/javascript/picker/tests/test_picker.js (+12/-12) lib/lp/app/javascript/picker/tests/test_picker_patcher.html (+1/-3) lib/lp/app/javascript/picker/tests/test_team.html (+1/-1) lib/lp/app/javascript/subscribers/tests/test_subscribers_list.html (+1/-1) lib/lp/app/javascript/tests/test_multicheckboxwidget.html (+1/-1) lib/lp/app/javascript/ui/ui.js (+21/-22) lib/lp/bugs/javascript/bug_tags_entry.js (+1/-2) lib/lp/bugs/javascript/bugtask_index.js (+2/-2) lib/lp/bugs/javascript/official_bug_tags.js (+2/-3) lib/lp/bugs/javascript/tests/test_bug_picker.html (+1/-1) lib/lp/bugs/javascript/tests/test_bugtask_delete.html (+1/-1) lib/lp/bugs/javascript/tests/test_duplicates.html (+1/-1) lib/lp/bugs/javascript/tests/test_information_type_choice.html (+1/-1) lib/lp/bugs/javascript/tests/test_official_bug_tags.html (+1/-1) lib/lp/bugs/javascript/tests/test_subscribers.html (+1/-1) lib/lp/code/javascript/tests/test_branchmergeproposal.nominate.html (+1/-1) lib/lp/code/javascript/tests/test_branchrevisionexpander.html (+1/-1) lib/lp/code/javascript/tests/test_bugspeclinks.html (+1/-1) lib/lp/code/javascript/tests/test_information_type_choice.html (+1/-1) lib/lp/registry/javascript/distroseries/tests/test_differences.html (+1/-1) lib/lp/registry/javascript/distroseries/tests/test_initseries.html (+1/-1) lib/lp/registry/javascript/distroseries/tests/test_widgets.html (+1/-1) lib/lp/registry/javascript/sharing/tests/test_granteepicker.html (+1/-1) lib/lp/registry/javascript/sharing/tests/test_granteetable.html (+1/-1) lib/lp/registry/javascript/sharing/tests/test_pillarsharingview.html (+1/-1) lib/lp/registry/javascript/tests/test_distroseriesdifferences_details.html (+1/-1) lib/lp/translations/javascript/tests/test_importqueue.html (+1/-1) |
| To merge this branch: | bzr merge lp:~jcsackett/launchpad/kill-lazr-1 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Curtis Hovey (community) | code | 2012-08-23 | Approve on 2012-08-23 |
|
Review via email:
|
|||
Commit Message
Migrates lazr.base to lp.ui
Description of the Change
Summary
=======
As the first part of migrating lazr to a new lp.ui namespace, this moves the
the lazr.base code to lp.ui.
Preimp
======
Spoke with Rick Harding
Implementation
==============
* Moved lib/lp/
and updated related paths.
* Renamed the lazr.ui namespace to lp.ui and updated callsites.
* Renamed lazr.base module to lp.ui-base module, and updated callsites.
Tests
=====
bin/test -vvc --layer=YUI
QA
==
Make sure javascript is still working across the site. This is a big one.
LoC
===
This removes 5 LoC; truthfully, it's LoC neutral if restricted to meaningful LoC.
Lint
====
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
buildout-
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
| j.c.sackett (jcsackett) wrote : | # |
> I see that
> lib/lp/
> lib/lp/
> Is this because they imported lazr.base, but they did not need to?
Correct; I think at some point they were using lazr.base, but a refactor removed the use. Alternatively, it was just an overzealous "require" statement.

This looks good to land. I see that lib/lp/ app/javascript/ choiceedit/ choiceedit. js and lib/lp/ bugs/javascript /bug_tags_ entry.js both do not change to "lp.ui-base". Is this because they imported lazr.base, but they did not need to?