Index: openacs-4/packages/acs-automated-testing/www/admin/index.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/acs-automated-testing/www/admin/index.adp,v
diff -u -r1.21.2.7 -r1.21.2.8
--- openacs-4/packages/acs-automated-testing/www/admin/index.adp	2 Sep 2019 13:17:35 -0000	1.21.2.7
+++ openacs-4/packages/acs-automated-testing/www/admin/index.adp	2 Sep 2019 14:41:34 -0000	1.21.2.8
@@ -1,10 +1,10 @@
 <master>
 <property name="doc(title)">@title;literal@</property>
-  <table border="0" cellspacing="0" cellpadding="3" width="100%">
+  <table class="main-table">
     <tr>
-      <th style="background-color:#ECECEC">Category</th>
-      <th style="background-color:#ECECEC">Mode</th>
-      <th style="background-color:#ECECEC">View by</th>
+      <th class="main-table-header">Category</th>
+      <th class="main-table-header">Mode</th>
+      <th class="main-table-header">View by</th>
     </tr>
     <tr>
 
@@ -23,39 +23,39 @@
         </else>
       </multiple> ]
  <div class="form-help-text">
-                  <img src="/shared/images/info.gif" width="12" height="9" alt="[i]" title="Help text" style="border:0">
+                  <img src="/shared/images/info.gif" width="12" height="9" alt="[i]" title="Help text" class="info-icon">
                   View and run only tests in this category.  Tests can
       have more than one category.  (<a
       href="/api-doc/proc-view?proc=aa%5fregister%5fcase">more info</a>)
                 </div>
 
 <p><if @stress;literal@ true>                
-      <a href="index?stress=0&amp;security_risk=@security_risk@&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" style="padding-top: 2px; padding-bottom: -2px;">
-        <img src="/resources/acs-subsite/checkboxchecked.gif" style="border:0" height="13" width="13">
+      <a href="index?stress=0&amp;security_risk=@security_risk@&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" class="checkbox-on">
+        <img src="/resources/acs-subsite/checkboxchecked.gif" class="checkbox-icon">
       </a>
     </if>
     <else>
-      <a href="index?stress=1&amp;security_risk=@security_risk@&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" style="padding: 0px;">
-        <img src="/resources/acs-subsite/checkbox.gif" alt="checkbox" style="border:0" height="13" width="13">
+      <a href="index?stress=1&amp;security_risk=@security_risk@&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" class="checkbox-off">
+        <img src="/resources/acs-subsite/checkbox.gif" alt="checkbox" class="checkbox-icon">
       </a>
     </else>
     Include Stress tests
 </p>
 <p><if @security_risk;literal@ true>                
-      <a href="index?stress=@stress@&amp;security_risk=0&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" style="padding-top: 2px; padding-bottom: -2px;">
-        <img src="/resources/acs-subsite/checkboxchecked.gif" style="border:0" height="13" width="13">
+      <a href="index?stress=@stress@&amp;security_risk=0&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" class="checkbox-on">
+        <img src="/resources/acs-subsite/checkboxchecked.gif" class="checkbox-icon">
       </a>
     </if>
     <else>
-      <a href="index?stress=@stress@&amp;security_risk=1&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" style="padding: 0px;">
-        <img src="/resources/acs-subsite/checkbox.gif" alt="checkbox" style="border:0" height="13" width="13">
+      <a href="index?stress=@stress@&amp;security_risk=1&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0" class="checkbox-off">
+        <img src="/resources/acs-subsite/checkbox.gif" alt="checkbox" class="checkbox-icon">
       </a>
     </else>
     Include tests that may compromise security
 </p>
     </td>
 
-    <td align="center" valign="top"> [
+    <td class="main-table-modes"> [
       <if @quiet;literal@ true>
          <strong> quiet </strong> | 
          <a href="index?stress=@stress@&amp;security_risk=@security_risk@&amp;by_package_key=@by_package_key@&amp;view_by=@view_by@&amp;by_category=@by_category@&amp;quiet=0">verbose</a>
@@ -64,12 +64,12 @@
          | <strong> verbose </strong>
       </else>  ]
  <div class="form-help-text">
-                  <img src="/shared/images/info.gif" width="12" height="9" alt="[i]" title="Help text" style="border:0">
+                  <img src="/shared/images/info.gif" width="12" height="9" alt="[i]" title="Help text" class="info-icon">
                   Quiet mode shows only test failures.
                 </div>
     </td>
 
-    <td align="center" valign="top"> [
+    <td class="main-table-modes"> [
       <if @view_by@ eq "" or @view_by@ eq "package">
          <strong> package </strong> | 
          <a href="index?stress=@stress@&amp;security_risk=@security_risk@&amp;by_package_key=@by_package_key@&amp;view_by=testcase&amp;by_category=@by_category@&amp;quiet=@quiet@">testcase</a>
@@ -100,8 +100,8 @@
     </blockquote>
     <form id="bulk-actions-form" action="">
       @bulk_actions_vars;literal@
-    <table cellpadding="2px">
-    <tr style="background-color:#c0c0c0">
+    <table class="package-table">
+    <tr class="package-table-header">
         <th>
           <input data-toggle="true" type="checkbox" checked="true" id="toggle-all"/>
           <script <if @::__csp_nonce@ not nil> nonce="@::__csp_nonce;literal@"</if>>
@@ -133,29 +133,29 @@
         <td><input type="checkbox" checked="true" name="package_key" value="@packageinfo.key@"/></td>
         <td> <a href="index?stress=@stress@&amp;security_risk=@security_risk@&amp;by_package_key=@packageinfo.key@&amp;view_by=testcase&amp;quiet=@quiet@">@packageinfo.key@</a></td>
         <if @packageinfo.total;literal@ eq 0>
-          <td align="right">No data</td>
-          <td align="right">-</td>
-          <td align="right">-</td>
-          <td align="right">-</td>
-          <td align="right">-</td>
-          <td align="right" class="proc_coverage_cell @packageinfo.proc_coverage_level@"><a href=proc-coverage?package_key=@packageinfo.key@>@packageinfo.proc_coverage@%</a></td>
+          <td class="package-table-cell">No data</td>
+          <td class="package-table-cell">-</td>
+          <td class="package-table-cell">-</td>
+          <td class="package-table-cell">-</td>
+          <td class="package-table-cell">-</td>
+          <td class="proc_coverage_cell @packageinfo.proc_coverage_level@"><a href=proc-coverage?package_key=@packageinfo.key@>@packageinfo.proc_coverage@%</a></td>
         </if><else>
-          <td align="right"> @packageinfo.total@ </td>
-          <td align="right"> @packageinfo.passes@ </td>
-          <td align="right"> @packageinfo.fails@ </td>
-          <td align="right"> @packageinfo.warnings@ </td>
-          <td align="right">
+          <td class="package-table-cell"> @packageinfo.total@ </td>
+          <td class="package-table-cell"> @packageinfo.passes@ </td>
+          <td class="package-table-cell"> @packageinfo.fails@ </td>
+          <td class="package-table-cell"> @packageinfo.warnings@ </td>
+          <td class="package-table-cell">
             <if @packageinfo.fails@ gt 0>
-               <span style="background-color: red; color: white; font-weight: bold;">FAILED</span>
+               <span class="result-failed">FAILED</span>
             </if>
             <elseif @packageinfo.warnings@ gt 0>
-              <span style="background-color: yellow; font-weight: bold;">WARNING</span>
+              <span class="result-warning">WARNING</span>
             </elseif>
             <else>
               OK
             </else>
           </td>
-          <td align="right" class="proc_coverage_cell @packageinfo.proc_coverage_level@"><a href=proc-coverage?package_key=@packageinfo.key@>@packageinfo.proc_coverage@%</a></td>
+          <td class="proc_coverage_cell @packageinfo.proc_coverage_level@"><a href=proc-coverage?package_key=@packageinfo.key@>@packageinfo.proc_coverage@%</a></td>
         </else>
       </tr>
     </multiple>
@@ -200,7 +200,7 @@
      </if>
    </blockquote>
     <table width="100%">
-    <tr  style="background-color:#c0c0c0">
+    <tr class="package-table-header">
         <th>Package key</th>
         <th>Testcase id</th>
         <th>Categories</th>
@@ -213,7 +213,7 @@
     </tr>
     <multiple name="tests">
       <if @tests.marker;literal@ true>
-        <tr><td colspan="9" align="center" bgcolor="#c0f0c0"><strong>@tests.package_key@</strong></td></tr>
+        <tr><td colspan="9" class="package-table-header-package-key"><strong>@tests.package_key@</strong></td></tr>
       </if>
         <if @tests.rownum@ odd>
           <tr class="odd">
@@ -228,24 +228,24 @@
         <if @tests.timestamp@ eq "">
           <td>No data</td>
           <td>-</td>
-          <td align="right">-</td>
-          <td align="right">-</td>
+          <td class="package-table-cell">-</td>
+          <td class="package-table-cell">-</td>
         </if><else>
           <td>
             <if @tests.fails@ gt 0>
-               <span style="background-color: red; color: white; font-weight: bold; padding: 4px;">FAILED</span>
+               <span class="result-failed">FAILED</span>
             </if>
             <elseif @tests.warnings@ gt 0>
-               <span style="background-color: yellow; font-weight: bold; padding: 4px;">WARNING</span>
+               <span class="result-warning">WARNING</span>
             </elseif>
             <else>
               OK
             </else>
           </td>
           <td> @tests.timestamp@ </td>
-          <td align="right"> @tests.passes@ </td>
-          <td align="right"> @tests.fails@ </td>
-          <td align="right"> @tests.warnings@ </td>
+          <td class="package-table-cell"> @tests.passes@ </td>
+          <td class="package-table-cell"> @tests.fails@ </td>
+          <td class="package-table-cell"> @tests.warnings@ </td>
         </else>
       </tr>
     </multiple>
Index: openacs-4/packages/acs-automated-testing/www/admin/testcase.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/acs-automated-testing/www/admin/testcase.adp,v
diff -u -r1.25.2.2 -r1.25.2.3
--- openacs-4/packages/acs-automated-testing/www/admin/testcase.adp	3 Apr 2019 13:03:46 -0000	1.25.2.2
+++ openacs-4/packages/acs-automated-testing/www/admin/testcase.adp	2 Sep 2019 14:41:34 -0000	1.25.2.3
@@ -4,32 +4,32 @@
   <h1>@title@</h1>
   <div class="description">
     <dl>
-      <dt>Description:</dt><dd>@testcase_desc@</dd>
-      <dt>Defined in file:</dt><dd>@testcase_file@</dd>
-      <dt>Categories:</dt><dd>@testcase_cats@</dd>
+      <dt class="description-list">Description:</dt><dd>@testcase_desc@</dd>
+      <dt class="description-list">Defined in file:</dt><dd>@testcase_file@</dd>
+      <dt class="description-list">Categories:</dt><dd>@testcase_cats@</dd>
       <if @bug_blurb@ not nil>
-        <dt>Bugs:</dt><dd>This test case covers OpenACS bug number(s):
+        <dt class="description-list">Bugs:</dt><dd>This test case covers OpenACS bug number(s):
           @bug_blurb;noquote@</dd>
       </if>
       <if @proc_blurb@ not nil>  
-        <dt>Procs:</dt><dd>This test case covers OpenACS proc(s):
+        <dt class="description-list">Procs:</dt><dd>This test case covers OpenACS proc(s):
           @proc_blurb;noquote@</dd>
       </if>
       <if @url_blurb@ not nil>  
-        <dt>URLs:</dt><dd>This test case covers the following URLs:
+        <dt class="description-list">URLs:</dt><dd>This test case covers the following URLs:
           @url_blurb;noquote@</dd>
       </if>
       <if @testcase_inits@ ne "">
-        <dt>Initialization Classes:</dt><dd>@testcase_inits@</dd>
+        <dt class="description-list">Initialization Classes:</dt><dd>@testcase_inits@</dd>
       </if>
       <if @fails@ gt 0 and @testcase_on_error@ ne "">
-        <dt class="fail">Testcase failure error response:</dt>
+        <dt class="description-list fail">Testcase failure error response:</dt>
         <dd>@testcase_on_error;noquote@</dd>
       </if>
       <if @showsource;literal@ true>
         <multiple name="bodys">
-	  <if @bodys:rowcount;literal@ lt 2><dt>Body:</dt></if>
-	  <else><dt>Body (part @bodys.body_number@)</dt></else>
+	  <if @bodys:rowcount;literal@ lt 2><dt class="description-list">Body:</dt></if>
+	  <else><dt class="description-list">Body (part @bodys.body_number@)</dt></else>
           <dd><pre class="code">@bodys.body;literal@</pre></dd>
         </multiple>
       </if>
@@ -66,8 +66,8 @@
 
   <table>
     <tr>
-      <th>Result</th>
-      <th>Count</th>
+      <th class="testcase-table-header">Result</th>
+      <th class="testcase-table-header">Count</th>
     </tr>
     <multiple name="tests_quiet">
       <tr>
@@ -87,9 +87,9 @@
 
   <table class="testlog">
     <tr>
-      <th class="timestamp">Time</th>
-      <th class="result">Result</th>
-      <th class="notes">Notes</th>
+      <th class="testcase-table-header timestamp">Time</th>
+      <th class="testcase-table-header result">Result</th>
+      <th class="testcase-table-header notes">Notes</th>
     </tr>
     <if @tests:rowcount;literal@ eq 0>
       <tr><td> No results </td></tr>
Index: openacs-4/packages/acs-automated-testing/www/admin/testcase.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/acs-automated-testing/www/admin/testcase.tcl,v
diff -u -r1.22.2.2 -r1.22.2.3
--- openacs-4/packages/acs-automated-testing/www/admin/testcase.tcl	30 Apr 2019 18:55:26 -0000	1.22.2.2
+++ openacs-4/packages/acs-automated-testing/www/admin/testcase.tcl	2 Sep 2019 14:41:34 -0000	1.22.2.3
@@ -21,6 +21,8 @@
 set title "Test case $testcase_id"
 set context [list $title]
 
+template::head::add_css -href /resources/acs-automated-testing/tests.css
+
 if {$quiet} {
     set filter { and result = 'fail'}
 } else {
@@ -145,70 +147,7 @@
 
 set quiet_url "[export_vars -base testcase -entire_form -exclude {quiet}]&quiet=1"
 set verbose_url "[export_vars -base testcase -entire_form -exclude {quiet}]&quiet=0"
-template::head::add_style \
-    -style {
-        .description h2 { 1.5em; }
-        div.description dd {
-            margin-left: 30px;
-            margin-right: 30px;
-        }
-        div.description dt {
-            margin-top: 12px;
-        }
-        .fail {
-            font-weight: bold;
-            color: red;
-        }
-        .ok {
-            font-weight: bold;
-            color: green;
-        }
-        .warn {
-            color: darkmagenta;
-        }
-        .sect {
-            font-size: large;
-            font-weight: bold;
-        }
 
-        td.log {
-            font-size: normal;
-            color: darkslateblue;
-            /*font-family: monospace;*/
-        }
-
-        dt {
-            font-weight: bold
-        }
-        th {
-            background: #c0c0c0;
-        }
-        table.testlog td code {
-            background-color: inherit;
-        }
-        table.testlog td code.ok {
-            color: forestgreen;
-            font-weight: normal;
-        }
-
-        td.timestamp {
-            white-space: nowrap;
-        }
-        
-        th.notes {
-            width: 70%;
-        }
-        td pre {
-            /*width: 70%;*/
-            margin: 0px;
-            border: 0px;
-            /*font-size: smaller;*/
-            padding: 2px;
-            background-color: inherit;
-        }
-        div.vl {border-left: 1px solid #aaa; padding:4px;height:3ex; display: inline-block;}
-    }
-
 ad_return_template
 
 # Local variables:
Index: openacs-4/packages/acs-automated-testing/www/resources/tests.css
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/acs-automated-testing/www/resources/tests.css,v
diff -u -r1.1.2.4 -r1.1.2.5
--- openacs-4/packages/acs-automated-testing/www/resources/tests.css	30 Aug 2019 17:53:48 -0000	1.1.2.4
+++ openacs-4/packages/acs-automated-testing/www/resources/tests.css	2 Sep 2019 14:41:34 -0000	1.1.2.5
@@ -1,3 +1,127 @@
+/*
+ * Automated testing main CSS
+ */
+
+/* Admin index styles */
+.main-table {
+    border: 0;
+    border-spacing: 0;
+    padding: 3px;
+    border-collapse: collapse;
+    width: 100%;
+}
+.main-table-header {
+    background-color: #ECECEC;
+}
+.main-table-modes {
+    text-align: center;
+    vertical-align: top;
+}
+.info-icon {
+    border: 0;
+}
+.checkbox-icon {
+    border: 0;
+    height: 13;
+    width: 13;
+}
+.checkbox-on {
+    padding-top: 2px;
+    padding-bottom: -2px;
+}
+.checkbox-off {
+    padding: 0px;
+}
+.package-table {
+    border-spacing: 3px;
+    border-collapse: separate;
+}
+table.package-table td {
+    padding: 3px;
+}
+.package-table-header {
+    background-color: #C0C0C0;
+}
+.package-table-header-package-key {
+    background-color: #C0F0C0;
+    text-align: center;
+}
+.package-table-cell {
+    text-align: right;
+}
+.result-failed {
+    background-color: red;
+    font-weight: bold;
+    color: white;
+}
+.result-warning {
+    background-color: yellow;
+    font-weight: bold;
+}
+
+/* Testcase */
+.description h2 { 1.5em; }
+div.description dd {
+    margin-left: 30px;
+    margin-right: 30px;
+}
+div.description dt {
+    margin-top: 12px;
+}
+.fail {
+    font-weight: bold;
+    color: red;
+}
+.ok {
+    font-weight: bold;
+    color: green;
+}
+.warn {
+    color: darkmagenta;
+}
+.sect {
+    font-size: large;
+    font-weight: bold;
+}
+td.log {
+    font-size: normal;
+    color: darkslateblue;
+    /*font-family: monospace;*/
+}
+dt.description-list {
+    font-weight: bold
+}
+.testcase-table-header {
+    background-color: #C0C0C0;
+}
+table.testlog td code {
+    background-color: inherit;
+}
+table.testlog td code.ok {
+    color: forestgreen;
+    font-weight: normal;
+}
+td.timestamp {
+    white-space: nowrap;
+}
+th.notes {
+    width: 70%;
+}
+td pre {
+    /*width: 70%;*/
+    margin: 0px;
+    border: 0px;
+    /*font-size: smaller;*/
+    padding: 2px;
+    background-color: inherit;
+}
+div.vl {
+    border-left: 1px solid #aaa;
+    padding:4px;
+    height:3ex;
+    display: inline-block;
+}
+
 /* Test coverage levels */
 .proc_coverage_cell > a {
     display: block;