Hi Barry,
again, a nice branch. Aside from my nitpick about the overly precise font size for .application-summary, I only noticed one "spurious change":
> === modified file 'lib/lp/registry/browser/person.py' > --- lib/lp/registry/browser/person.py 2009-08-20 16:13:15 +0000 > +++ lib/lp/registry/browser/person.py 2009-08-20 22:57:10 +0000 > @@ -1337,6 +1337,7 @@ > > > class PersonSetActionNavigationMenu(RegistryCollectionActionMenuBase): > +
please remove the empty line.
> """Action menu for `PeopleSearchView`.""" > usedfor = IPersonSet > links = ['register_team', 'register_project', 'create_account',
Abel
« Back to merge proposal
Hi Barry,
again, a nice branch. Aside from my nitpick about the overly precise font size for .application- summary, I only noticed one "spurious change":
> === modified file 'lib/lp/ registry/ browser/ person. py' registry/ browser/ person. py 2009-08-20 16:13:15 +0000 registry/ browser/ person. py 2009-08-20 22:57:10 +0000 NavigationMenu( RegistryCollect ionActionMenuBa se):
> --- lib/lp/
> +++ lib/lp/
> @@ -1337,6 +1337,7 @@
>
>
> class PersonSetAction
> +
please remove the empty line.
> """Action menu for `PeopleSearchVi ew`."""
> usedfor = IPersonSet
> links = ['register_team', 'register_project', 'create_account',
Abel