launchpad-reviewers team mailing list archive

[Paul Hummer <paul@xxxxxxxxxxxxxxxxxxxx>]

Paul Hummer has proposed merging lp:~rockstar/launchpad/person-recipe-listings into lp:launchpad/devel.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  #592937 "View source package recipes" oddly located on Person index
  https://bugs.launchpad.net/bugs/592937


This branch is really easy.  It moves a person's recipe links to be part of the ppa portlet instead of where it is now.  It fixes bug #592937.
-- 
https://code.launchpad.net/~rockstar/launchpad/person-recipe-listings/+merge/31688
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rockstar/launchpad/person-recipe-listings into lp:launchpad/devel.

=== modified file 'lib/lp/registry/templates/person-portlet-contact-details.pt'
--- lib/lp/registry/templates/person-portlet-contact-details.pt	2010-06-24 14:10:56 +0000
+++ lib/lp/registry/templates/person-portlet-contact-details.pt	2010-08-03 20:56:57 +0000
@@ -188,18 +188,13 @@
     </dl>
 
     <dl id="karma">
-      <dt>Karma:</dt>
+      <dt>earma:</dt>
       <dd><a id="karma-total"
              tal:attributes="href overview_menu/karma/url"
              tal:content="context/karma">342</a>
       </dd>
     </dl>
   </div>
-  <div tal:define="link context/menu:overview/view_recipes"
-       tal:condition="link/enabled">
-    <a tal:replace="structure link/fmt:link" />
-  </div>
-
 
 </div>
 

=== modified file 'lib/lp/soyuz/templates/person-portlet-ppas.pt'
--- lib/lp/soyuz/templates/person-portlet-ppas.pt	2010-04-19 03:36:55 +0000
+++ lib/lp/soyuz/templates/person-portlet-ppas.pt	2010-08-03 20:56:57 +0000
@@ -21,4 +21,10 @@
 
     </ul>
   </div>
+  <div tal:define="link context/menu:overview/view_recipes"
+       tal:condition="link/enabled">
+    <a tal:replace="structure link/fmt:link" />
+  </div>
+
+
 </tal:root>