launchpad-reviewers team mailing list archive

[Huw Wilkins <huw.wilkins@xxxxxxxxxxxxx>]

Huw Wilkins has proposed merging lp:~huwshimi/launchpad/remove-participation into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~huwshimi/launchpad/remove-participation/+merge/66458

The Participation portlet on user profiles is redundant as it duplicates the main navigation and also adds ui clutter (and possible confusion).

This branch removes the portlet altogether.
-- 
https://code.launchpad.net/~huwshimi/launchpad/remove-participation/+merge/66458
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~huwshimi/launchpad/remove-participation into lp:launchpad.

=== modified file 'lib/lp/registry/templates/person-index.pt'
--- lib/lp/registry/templates/person-index.pt	2010-11-16 15:36:07 +0000
+++ lib/lp/registry/templates/person-index.pt	2011-06-30 13:09:39 +0000
@@ -50,34 +50,6 @@
   <metal:contact use-macro="context/@@+person-macros/contact">
     Contact this person
   </metal:contact>
-
-  <div id="involvement" class="portlet involvement"
-    tal:condition="context/is_valid_person">
-    <h2>Participation</h2>
-
-    <ul>
-      <li>
-        <a class="sprite bugs"
-          tal:attributes="href context/fmt:url:bugs">Bugs</a>
-      </li>
-      <li>
-        <a class="sprite answers"
-          tal:attributes="href context/fmt:url:answers">Answers</a>
-      </li>
-      <li>
-        <a class="sprite translations"
-          tal:attributes="href context/fmt:url:translations">Translations</a>
-      </li>
-      <li>
-        <a class="sprite code"
-          tal:attributes="href context/fmt:url:code">Code</a>
-      </li>
-      <li>
-        <a class="sprite blueprints"
-          tal:attributes="href context/fmt:url:blueprints">Blueprints</a>
-      </li>
-    </ul>
-  </div>
 </div>
 
 <div metal:fill-slot="main"