Merge ~robru/autopkgtest-cloud:master into autopkgtest-cloud:master

Proposed by Robert Bruce Park on 2017-04-06
Status: Merged
Merged at revision: ff6cd1dd9ba4d4085ec3b4adcac12f35827cfdc4
Proposed branch: ~robru/autopkgtest-cloud:master
Merge into: autopkgtest-cloud:master
Diff against target: 31 lines (+12/-1)
2 files modified
webcontrol/static/style.css (+4/-0)
webcontrol/templates/browse-results.html (+8/-1)
Reviewer Review Type Date Requested Status
Iain Lane 2017-04-06 Approve on 2017-04-06
Review via email: mp+322132@code.launchpad.net

Description of the Change

Style fix that prevents the retry link from wrapping in an ugly way

To post a comment you must log in.
Robert Bruce Park (robru) wrote :

Pages with long triggers get squished and wrap in an ugly way:

http://autopkgtest.ubuntu.com/packages/l/linux-hwe/xenial/i386

Iain Lane (laney) wrote :

Ta

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/webcontrol/static/style.css b/webcontrol/static/style.css
2index 749f222..0875b7c 100644
3--- a/webcontrol/static/style.css
4+++ b/webcontrol/static/style.css
5@@ -38,3 +38,7 @@
6 background-color: #E90051;
7 color: white;
8 }
9+
10+.nowrap {
11+ white-space: nowrap;
12+}
13diff --git a/webcontrol/templates/browse-results.html b/webcontrol/templates/browse-results.html
14index 059b4ec..bd7b61f 100644
15--- a/webcontrol/templates/browse-results.html
16+++ b/webcontrol/templates/browse-results.html
17@@ -19,7 +19,14 @@
18 <td>{{row[2]}}</td>
19 <td>{{row[3]}}</td>
20 <td class="{{row[4]}}">{{row[4]}}</td>
21- <td><a href="{{row[5]}}/log.gz">log</a> &emsp; <a href="{{row[5]}}/artifacts.tar.gz">artifacts</a></td>
22+ <td class="nowrap">
23+ <a href="{{row[5]}}/log.gz">log</a> &emsp;
24+ <a href="{{row[5]}}/artifacts.tar.gz">artifacts</a> &emsp;
25+ {% if row[4] == 'fail' %}
26+ {% set ts = row[1].split()|join("&trigger=") %}
27+ <a href="{{base_url}}request.cgi?release={{release}}&arch={{arch}}&package={{package}}&trigger={{ts}}">retry</a>
28+ {% endif %}
29+ </td>
30 </tr>
31 {% endfor %}
32

Subscribers

People subscribed via source and target branches