launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Diff comments:

> diff --git a/lib/lp/soyuz/templates/distroseries-queue.pt b/lib/lp/soyuz/templates/distroseries-queue.pt
> index 39629ce..3850174 100644
> --- a/lib/lp/soyuz/templates/distroseries-queue.pt
> +++ b/lib/lp/soyuz/templates/distroseries-queue.pt
> @@ -147,10 +147,14 @@
>                  </td>
>               </tr>
>             </tbody>
> -           <tbody tal:attributes="class string:${filelist_class}">
> +           <tbody tal:attributes="class string:${filelist_class}"
> +                  tal:define="hasActions python: len(view.availableActions()) > 0">
>               <metal:filelist use-macro="template/macros/package-filelist"/>
> +
>               <tr class="log-content" tal:repeat="log packageupload/logs">
> -               <td colspan="2" style="border: 0"></td>
> +               <td tal:attributes="colspan python: 2 if hasActions else 1"
> +                   style="border: 0"></td>
> +
>                 <td colspan="8" style="border: 0">

Doesn't this technically need adjustment too, so that it's 8 if hasActions else 9?

>                   <span tal:content="log/new_status"></span>
>                   <span tal:attributes="title log/date_created/fmt:datetime"


-- 
https://code.launchpad.net/~pappacena/launchpad/+git/launchpad/+merge/378679
Your team Launchpad code reviewers is requested to review the proposed merge of ~pappacena/launchpad:pgk-upload-log-positioning into launchpad:master.