launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging lp:~cjwatson/launchpad/tidy-dsp-index into lp:launchpad.

Commit message:
Move "Latest upload:" onto the same row as "Architectures:".

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/tidy-dsp-index/+merge/270301

Move "Latest upload:" onto the same row as "Architectures:" on DSP:+index.

https://code.launchpad.net/~cjwatson/launchpad/delete-dsp-component-ui/+merge/270169 deleted "Component:", but I forgot to check whether the table would be reflowed automatically, and it won't, so we end up with an upside-down-L-shaped table of package information on https://qastaging.launchpad.net/ubuntu/+source/man-db which I think looks a bit odd.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~cjwatson/launchpad/tidy-dsp-index into lp:launchpad.

=== modified file 'lib/lp/registry/templates/distributionsourcepackage-index.pt'
--- lib/lp/registry/templates/distributionsourcepackage-index.pt	2015-09-04 12:36:43 +0000
+++ lib/lp/registry/templates/distributionsourcepackage-index.pt	2015-09-07 12:48:57 +0000
@@ -75,11 +75,11 @@
             <dt>Urgency:<sup>*</sup></dt>
             <dd tal:content="current/urgency/title"/>
           </dl>
-          <dl>
+          <dl style="clear:both">
             <dt>Architectures:<sup>*</sup></dt>
             <dd tal:content="current/architecturehintlist"/>
           </dl>
-          <dl style="clear:both">
+          <dl>
             <dt>Latest upload:</dt>
             <dd><a tal:content="current/version"
                    tal:attributes="href current/fmt:url">3.2.1-3</a>