Bug 289594 - The table on the email settings page no longer is 100% wide. It also now supports even/odd row styling.

r=gerv, a=justdave

4 files changed, 16 insertions, 11 deletions

diff --git a/skins/standard/buglist.css b/skins/standard/buglist.css
index f1f1d74dc..45e4937d9 100644
--- a/skins/standard/buglist.css
+++ b/skins/standard/buglist.css
@@ -32,14 +32,6 @@ table#form tr th { text-align: right; }
 table.bz_buglist td, table.bz_buglist th { 
 }
 
-/* For styling rows; by default striped white and gray */
-tr.bz_odd {
-    background-color: #F7F7F7;
-}
-
-tr.bz_even {
-}
-
 /* we use a first-child class and not the pseudo-class because IE
  * doesn't support it :-( */
 tr.bz_secure td.first-child { 
diff --git a/skins/standard/global.css b/skins/standard/global.css
index 724a75f11..0f274ea56 100644
--- a/skins/standard/global.css
+++ b/skins/standard/global.css
@@ -293,6 +293,18 @@ body
 .bz_private { color: darkred ; background : #f3eeee ; }
 .bz_disabled { color: #a0a0a0 ; }
 
+.bz_row_odd
+{
+    background-color: #CCCCCC;
+    color: #000000;
+}
+
+.bz_row_even
+{
+    background-color: #FFFFFF;
+    color: #000000;
+}
+
 .bz_comment { background-color: #e0e0e0; }
 .bz_comment_hilite pre
 {
diff --git a/template/en/default/account/prefs/email.html.tmpl b/template/en/default/account/prefs/email.html.tmpl
index f712bd8fa..787cc8dfb 100644
--- a/template/en/default/account/prefs/email.html.tmpl
+++ b/template/en/default/account/prefs/email.html.tmpl
@@ -148,7 +148,7 @@ document.write('<input type="button" value="Disable All Mail" onclick="SetCheckb
       description = "Voter" },
 ] %]
 
-<table width="100%" border="1">
+<table class="bz_emailprefs" border="1">
   <tr>
     <td colspan="[% (useqacontact AND usevotes) ? '5' : 
                      ((useqacontact OR usevotes) ? '4' : '3') %]" 
@@ -171,7 +171,8 @@ document.write('<input type="button" value="Disable All Mail" onclick="SetCheckb
   </tr>
   
   [% FOREACH event = events %]  
-    <tr>
+    [% count = loop.count() %]
+    <tr class="bz_row_[% count % 2 == 1 ? "odd" : "even" %]">
       [% FOREACH relationship = relationships %]
       [% NEXT IF (relationship.id == constants.REL_QA AND NOT useqacontact) OR
                  (relationship.id == constants.REL_VOTER AND NOT usevotes) %]
diff --git a/template/en/default/list/table.html.tmpl b/template/en/default/list/table.html.tmpl
index 1a95fecd3..9ba10b6ec 100644
--- a/template/en/default/list/table.html.tmpl
+++ b/template/en/default/list/table.html.tmpl
@@ -161,7 +161,7 @@
              [%+ "bz_$bug.resolution" FILTER css_class_quote IF bug.resolution -%]
              [%+ "bz_secure" IF bug.secure_mode -%]
              [%+ "bz_secure_mode_$bug.secure_mode" FILTER css_class_quote IF bug.secure_mode -%]
-             [%+ count % 2 == 1 ? "bz_odd" : "bz_even" -%]
+             [%+ count % 2 == 1 ? "bz_row_odd" : "bz_row_even" -%]
              ">