From dbdb5de3a403d523bc70c10cb83969418c1b8b3f Mon Sep 17 00:00:00 2001
From: Jeremy Harris <jgh@wizmail.org>
Date: Fri, 30 Dec 2016 09:59:14 +0000
Subject: [PATCH] Give whole range as title on colour-bar

---
 templates/history.tt | 2 +-
 templates/status.tt  | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/templates/history.tt b/templates/history.tt
index 2281a3f..442a4c1 100644
--- a/templates/history.tt
+++ b/templates/history.tt
@@ -46,7 +46,7 @@ See accompanying License file for license details
 -%]
 [%# The IF-backstop is ugly and I'm not understanding why the res_img lookup does not for for '.' -%]
 [%- BLOCK colour ; IF res_imgs.$c.defined ; res_imgs.$c ; ELSE %]/img/t_notdone.gif [% END; END %]
-[%- BLOCK colourbar %]<img src="[% PROCESS colour %]" title="[% st %]" alt="[% c %]" height="16" width="[% w %]" class="inline" align="bottom" />[%- END -%]
+[%- BLOCK colourbar %]<img src=[% PROCESS colour %] title="[% st %]-[% st + w %]" height=16 width=[% w %] class=inline align=bottom/>[%- END -%]
 [% BLOCK stdet %]
     <tr [% PROCESS cl bgfor=row.stage %]>
       <td>[%- row.when_ago | replace('\s','&nbsp;') %]&nbsp;ago&nbsp;</td>
diff --git a/templates/status.tt b/templates/status.tt
index 29be4e3..8e44bfc 100644
--- a/templates/status.tt
+++ b/templates/status.tt
@@ -36,7 +36,7 @@ See accompanying License file for license details
      '.' => '/img/t_notdone.gif',
      }
 -%]
-[%- BLOCK img ; IF flag == 'depend' or flag == 'gnu-ld' ; ; ELSIF flag_imgs.$flag %]<img src="[% flag_imgs.$flag %]" title="[% flag %]" alt="[% flag %]" height="16" width="16" class="inline" align="bottom" />  [% ELSE %][%#
+[%- BLOCK img ; IF flag == 'depend' or flag == 'gnu-ld' ; ; ELSIF flag_imgs.$flag %]<img src=[% flag_imgs.$flag %] title="[% flag %]" alt="[% flag %]" height=16 width=16 class=inline align=bottom/>  [% ELSE %][%#
 																									  flag ; ' '
 %][% END ; END -%]
 [%- BLOCK sysnotes ; IF row.sys_notes %]<img src="/img/notes.png" height="16" width="16" title="[% row.sys_notes_ts.replace(' .*','') | html %]: [% row.sys_notes | html %]" />
@@ -46,7 +46,7 @@ See accompanying License file for license details
 [%- END -%]
 [%# The IF-backstop is ugly and I'm not understanding why the res_img lookup does not for for '.' -%]
 [%- BLOCK colour ; IF res_imgs.$c.defined ; res_imgs.$c ; ELSE %]/img/t_notdone.gif [% END; END %]
-[%- BLOCK colourbar %]<img src="[% PROCESS colour %]" title="[% st %]" alt="[% c %]" height="16" width="[% w %]" class="inline" align="bottom" />[%- END -%]
+[%- BLOCK colourbar %]<img src=[% PROCESS colour %] title="[% st %]-[% st + w %]" height=16 width=[% w %] class=inline align=bottom/>[%- END -%]
 [% WRAPPER 'page.tt' 
 	title = 'Exim BuildFarm Status'
 	bodyclass = 'none'
@@ -63,7 +63,7 @@ See accompanying License file for license details
     </p>
 <table><tr><th class="head" rowspan="2">Legend</th>
 [% FOREACH flagset IN flag_imgs %]
-<td><img src="[% flagset.value %]" title="[% flagset.key %]" alt="[% flagset.key %]" height="16" width="16" class="inline"  align="center"/> =  [% flagset.key %]</td>
+<td><img src=[% flagset.value %] title="[% flagset.key %]" alt="[% flagset.key %]" height=16 width=16 class=inline  align=center/> =  [% flagset.key %]</td>
 [% IF loop.count == 7 %]</tr><tr>[% END %]
 [% END %]
 </tr></table>
-- 
2.30.2