Fix for bug 245924: updates Bugzilla footer to take advantage of HTML4 and CSS formatting.

Patch by Vitaly Harisov.
r=myk
a=myk

4 files changed, 31 insertions, 41 deletions

diff --git a/template/en/default/global/banner.html.tmpl b/template/en/default/global/banner.html.tmpl
index 95a7605b9..42ccce1aa 100644
--- a/template/en/default/global/banner.html.tmpl
+++ b/template/en/default/global/banner.html.tmpl
@@ -18,6 +18,7 @@
   #
   # Contributor(s): Gervase Markham <gerv@gerv.net>
   #                 Matthew Tuck <matty@chariot.net.au>
+  #                 Vitaly Harisov  <vitaly@rathedg.com>
   #%]
 
 [%# Migration note: this file corresponds to the old Param 'bannerhtml' %]
diff --git a/template/en/default/global/footer.html.tmpl b/template/en/default/global/footer.html.tmpl
index 4bde601d8..fc1016621 100644
--- a/template/en/default/global/footer.html.tmpl
+++ b/template/en/default/global/footer.html.tmpl
@@ -17,6 +17,7 @@
   # Rights Reserved.
   #
   # Contributor(s): Gervase Markham <gerv@gerv.net>
+  #                 Svetlana Harisova <light@rathedg.com>
   #%]
 
 [%# INTERFACE:
@@ -30,13 +31,8 @@
   # 'footerhtml'
   #%]
 
-<table border="0">
-  <tr>
-    <td bgcolor="#000000" valign="top">
-      <table border="0" cellpadding="10" cellspacing="0" width="100%"
-             bgcolor="lightyellow">
-        <tr>
-          <td>
+<div id="footer">
+  <div class="intro"></div>
 
 [%# Migration note: the old param 'blurbhtml' goes here %]
 
@@ -44,11 +40,8 @@
 
            [% PROCESS "global/useful-links.html.tmpl" %]
 
-          </td>
-        </tr>
-      </table>
-    </td>
-  </tr>
-</table>
+  <div class="outro"></div>
+</div>
+
 </body>
 </html>
diff --git a/template/en/default/global/useful-links.html.tmpl b/template/en/default/global/useful-links.html.tmpl
index d238a4253..8e208927c 100644
--- a/template/en/default/global/useful-links.html.tmpl
+++ b/template/en/default/global/useful-links.html.tmpl
@@ -17,6 +17,7 @@
   # Rights Reserved.
   #
   # Contributor(s): Gervase Markham <gerv@gerv.net>
+  #                 Svetlana Harisova <light@rathedg.com>
   #%]
 
 [%# Migration note: this whole file corresponds to the old %commandmenu% 
@@ -25,18 +26,15 @@
 [% PROCESS global/variables.none.tmpl %]
 
 <form method="get" action="show_bug.cgi">
-  <table width="100%" cellspacing="4">
-    <tr>
-      <td>
-        Actions:
-      </td>
-      
-      <td valign="middle">
+<div id="useful-links">
+  <div class="group">
+    <div id="link-action">Actions:</div>
+    <div class="links">
         <a href="enter_bug.cgi">New</a> | 
         <a href="query.cgi">Search</a> |
         
-        <input type="submit" value="Find"> [% terms.bug %] # 
-        <input name="id" size="6"> | 
+        <input class="btn" type="submit" value="Find"> [% terms.bug %] # 
+        <input class="txt" name="id" size="6"> | 
         
         <a href="report.cgi">Reports</a> 
         
@@ -62,15 +60,14 @@
             [% END %]
             | <a href="query.cgi?GoAheadAndLogIn=1">Log&nbsp;In</a>
         [% END %]
-      </td>
-    </tr>
+    </div>
+  </div>
       
     [% IF user.login %] 
-    <tr>
-      <td>
-        Edit:
-      </td>
-      <td valign="middle">
+  <div class="group">
+    <div id="link-edit">Edit:</div>
+    <div class="links">
+
         <a href="userprefs.cgi">Prefs</a>
         [% ' | <a href="editparams.cgi">Parameters</a>' 
                                                   IF user.groups.tweakparams %]
@@ -84,21 +81,20 @@
                                                  IF user.groups.creategroups %]
         [% ' | <a href="editkeywords.cgi">Keywords</a>' 
                                                  IF user.groups.editkeywords %]
-      </td>
-    </tr> 
+    </div>
+  </div>
     [% END %]
     
 
     [%# Saved searches %]
     
-    <tr>
+  <div class="group">
+    <div id="link-saved">
       [% IF user.showmybugslink OR user.queries.size %]
-        <td>
           Saved&nbsp;Searches:
-        </td>
       [% END %]
-      
-      <td valign="middle">
+    </div>
+    <div class="links">
         [% IF user.showmybugslink %]
           [% filtered_username = user.login FILTER url_quote %]
           <a href="[% Param('mybugstemplate').replace('%userid%', filtered_username) %]">My&nbsp;[% terms.Bugs %]</a>
@@ -112,8 +108,7 @@
             [% print_pipe = 1 %]
           [% END %]
         [% END %]
-      </td>
-    </tr>
-    
-  </table>
+    </div>
+  </div>
+</div>
 </form>
diff --git a/template/en/default/search/search-specific.html.tmpl b/template/en/default/search/search-specific.html.tmpl
index 345483320..ca3dae67b 100644
--- a/template/en/default/search/search-specific.html.tmpl
+++ b/template/en/default/search/search-specific.html.tmpl
@@ -101,7 +101,8 @@ for "crash secure SSL flash".
     <td>
       <input type="submit" value="Search">
     </td>
-
+  </tr>
+</table>
 </form>
 
 [% PROCESS global/footer.html.tmpl %]