launchpad-reviewers team mailing list archive

[Steve Kowalik <stevenk@xxxxxxxxxx>]

Steve Kowalik has proposed merging lp:~stevenk/launchpad/branch-subscriber-vertical-space into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #524322 in Launchpad itself: "Excessive gaps in branch subscriber portlet"
  https://bugs.launchpad.net/launchpad/+bug/524322

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/branch-subscriber-vertical-space/+merge/66615

The branch subscriber portlet now uses the 'section' class, rather than 'actions',
-- 
https://code.launchpad.net/~stevenk/launchpad/branch-subscriber-vertical-space/+merge/66615
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/branch-subscriber-vertical-space into lp:launchpad.

=== modified file 'lib/lp/code/templates/branch-portlet-subscribers.pt'
--- lib/lp/code/templates/branch-portlet-subscribers.pt	2010-06-10 07:54:59 +0000
+++ lib/lp/code/templates/branch-portlet-subscribers.pt	2011-07-01 14:17:27 +0000
@@ -7,7 +7,7 @@
        tal:define="context_menu view/context/menu:context">
 
     <div>
-      <div class="actions">
+      <div class="section">
       <div
         tal:define="link context_menu/subscription"
         tal:condition="link/enabled"