Launchpad itself

Merge lp:~edwin-grubbs/launchpad/featured-projects-ui-3 into lp:launchpad

  1. featured-projects-ui-3
  2. Merge into devel
Proposed by Edwin Grubbs
Status: Merged
Approved by: Barry Warsaw
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~edwin-grubbs/launchpad/featured-projects-ui-3
Merge into: lp:launchpad
Diff against target: None lines
To merge this branch: bzr merge lp:~edwin-grubbs/launchpad/featured-projects-ui-3
Reviewer Review Type Date Requested Status
Barry Warsaw (community) code ui* Approve
Review via email: mp+12053@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Edwin Grubbs (edwin-grubbs) wrote :

Summary
-------

Convert root-featuredprojects.pt to UI 3.0.

Tests
-----

./bin/test -vv -t featuredproject.txt

Demo and Q/A
------------

* Open http://launchpad.dev/+featuredprojects

Revision history for this message
Barry Warsaw (barry) wrote :

Hi Edwin, thanks for working on this.

I'd prefer it if you wrote the code like this:

    label = 'foo'
    page_title = label

because eventually you will be able to get rid of the page_title line.

review: Approve (code ui*)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/canonical/launchpad/pagetitles.py'
--- lib/canonical/launchpad/pagetitles.py 2009-09-17 16:12:20 +0000
+++ lib/canonical/launchpad/pagetitles.py 2009-09-18 13:24:10 +0000
@@ -786,8 +786,6 @@
786786
787remotebug_index = ContextTitle('%s')787remotebug_index = ContextTitle('%s')
788788
789root_featuredprojects = 'Manage featured projects in Launchpad'
790
791root_index = 'Launchpad'789root_index = 'Launchpad'
792790
793rosetta_about = 'About Launchpad Translations'791rosetta_about = 'About Launchpad Translations'
794792
=== modified file 'lib/canonical/launchpad/templates/root-featuredprojects.pt'
--- lib/canonical/launchpad/templates/root-featuredprojects.pt 2009-07-17 17:59:07 +0000
+++ lib/canonical/launchpad/templates/root-featuredprojects.pt 2009-09-18 13:24:10 +0000
@@ -3,17 +3,12 @@
3 xmlns:tal="http://xml.zope.org/namespaces/tal"3 xmlns:tal="http://xml.zope.org/namespaces/tal"
4 xmlns:metal="http://xml.zope.org/namespaces/metal"4 xmlns:metal="http://xml.zope.org/namespaces/metal"
5 xmlns:i18n="http://xml.zope.org/namespaces/i18n"5 xmlns:i18n="http://xml.zope.org/namespaces/i18n"
6 xml:lang="en"6 metal:use-macro="view/macro:page/main_only"
7 lang="en"
8 dir="ltr"
9 metal:use-macro="view/macro:page/applicationhome"
10 i18n:domain="launchpad"7 i18n:domain="launchpad"
11>8>
129
13<body>10<body>
1411
15<h1 metal:fill-slot="heading">Manage featured projects</h1>
16
17<div metal:fill-slot="main">12<div metal:fill-slot="main">
1813
19 <div metal:use-macro="context/@@launchpad_form/form">14 <div metal:use-macro="context/@@launchpad_form/form">
2015
=== modified file 'lib/lp/registry/browser/featuredproject.py'
--- lib/lp/registry/browser/featuredproject.py 2009-06-25 04:06:00 +0000
+++ lib/lp/registry/browser/featuredproject.py 2009-09-18 13:24:10 +0000
@@ -41,6 +41,8 @@
41class FeaturedProjectsView(LaunchpadFormView):41class FeaturedProjectsView(LaunchpadFormView):
42 """A view for adding and removing featured projects."""42 """A view for adding and removing featured projects."""
4343
44 page_title = label = 'Manage featured projects in Launchpad'
45
44 schema = FeaturedProjectForm46 schema = FeaturedProjectForm
45 custom_widget('remove', LabeledMultiCheckBoxWidget)47 custom_widget('remove', LabeledMultiCheckBoxWidget)
4648