Merge lp:~milo/linaro-license-protection/bug1183747 into lp:~linaro-automation/linaro-license-protection/trunk

[Milo Casagrande]

The MP adds:
- an "align=center" for the License colum, now the text should be correctly aligned
- an onmouseover and onmouseout event on all the rows in the table (apart from the first one with the Parent directory link), so that the row is highlighted when passing over it
- fixes for the table header, the markup was not correct
- use of colspan instead of adding empty columns in a row

[Stevan Radaković (stevanr)]

Looks good.
Approve +1.

review: Approve

=== modified file 'templates/dir_template.html'
--- templates/dir_template.html	2013-05-10 09:44:09 +0000
+++ templates/dir_template.html	2013-08-21 16:35:48 +0000
@@ -3,24 +3,25 @@
 {% block content %}
 {{ header_content|safe }}
 <table>
-  <tr>
-    <th></th><th>Name</th><th>Last modified</th><th>Size</th><th>License</th>
-    <tr><th colspan="5"><hr></th>
-  </tr>
+    <thead>
+        <tr>
+            <th></th><th>Name</th><th>Last modified</th><th>Size</th><th>License</th>
+        </tr>
+        <tr>
+            <th colspan="5"><hr></th>
+        </tr>
+    </thead>
   {% if up_dir != None %}
     <tr>
       <td>
         <img src="{{ STATIC_URL }}/static/go-up.png" alt="Parent Directory"/>
       </td>
-      <td><a href="{{ up_dir }}">Parent Directory</a></td>
-      <td></td>
-      <td></td>
-      <td></td>
+      <td colspan="0"><a href="{{ up_dir }}">Parent Directory</a></td>
     </tr>
   {% endif %}
 
 {% for value in dirlist %}
-  <tr>
+  <tr onmouseover="this.style.background='#CBE68F'" onmouseout="this.style.background='#FFFFFF'">
     <td valign="top">
       <img src="{{ STATIC_URL }}/static/{{ value.type }}.png"
            alt="{{ value.type }}"/>
@@ -37,7 +38,7 @@
     <td align="center">
       {% if value.type != 'folder' %}{{ value.size }}{% endif %}
     </td>
-    <td>
+    <td align="center">
       {% if value.license_digest_list == "OPEN" %}
         <i>open</i>
       {% else %}