Bug 840407: Remove the "align" and "valign" attributes

r/a=justdave

9 files changed, 122 insertions, 31 deletions

diff --git a/skins/contrib/Dusk/global.css b/skins/contrib/Dusk/global.css
index f051a3b58..4f6e00341 100644
--- a/skins/contrib/Dusk/global.css
+++ b/skins/contrib/Dusk/global.css
@@ -66,12 +66,6 @@ hr {
 
 /* edit */
 
-#bugzilla-body th {
-    font-weight: bold;
-    vertical-align: top;
-    white-space: nowrap;
-}
-
 #bug-form td {
     padding-top: 2px;
 }
diff --git a/skins/standard/admin.css b/skins/standard/admin.css
index b5f36b654..9367e2a4f 100644
--- a/skins/standard/admin.css
+++ b/skins/standard/admin.css
@@ -43,10 +43,6 @@ p.areyoureallyreallysure {
     border: 1px solid black;
 }
 
-#admin_table th, #flag_types_bugs th, #flag_types_attachments th {
-    white-space: normal !important;
-}
-
 #admin_table th, #admin_table td,
 #flag_types_bugs th , #flag_types_bugs td,
 #flag_types_attachments th, #flag_types_attachments td {
@@ -88,12 +84,9 @@ td.admin_links dt.forbidden a, td.admin_links dd.forbidden a {
     cursor: default;
 }
 
-.col-header {
+.open-status, .closed-status {
     width: 8em;
-}
-
-.checkbox-cell {
-    border: 1px black solid;
+    padding: 0.3em;
 }
 
 /* Grey-green color */
@@ -175,7 +168,6 @@ table.whining_list th, table.whining_list td,
 table.whining_list th {
     text-align: right;
     font-weight: normal !important;
-    white-space: normal !important;
 }
 
 table.whining_list th.subtitle {
@@ -212,3 +204,12 @@ table.schedule_list th, table.search_list th {
 #mass-remove legend {
     white-space: nowrap;
 }
+
+#workflow {
+    border-collapse: collapse;
+}
+
+#workflow td {
+    border: 1px dotted black;
+    text-align: center;
+}
diff --git a/skins/standard/attachment.css b/skins/standard/attachment.css
index c765aa5a9..b08cb6136 100644
--- a/skins/standard/attachment.css
+++ b/skins/standard/attachment.css
@@ -207,8 +207,13 @@ div#update_container {
     margin-left: 2%;
 }
 
-.viewall_desc {
+.viewall_frame_container {
     width: 75%;
+    margin: 2em auto;
+}
+
+.viewall_desc {
+    width: 100%;
     border: 1px solid black;
     border-spacing: 0px;
     border-collapse: collapse;
@@ -220,7 +225,7 @@ div#update_container {
 }
 
 .viewall_frame {
-    width: 75%;
+    width: 100%;
     height: 350px;
 }
 
diff --git a/skins/standard/buglist.css b/skins/standard/buglist.css
index 75dafb9cf..963d82965 100644
--- a/skins/standard/buglist.css
+++ b/skins/standard/buglist.css
@@ -5,7 +5,8 @@
  * This Source Code Form is "Incompatible With Secondary Licenses", as
  * defined by the Mozilla Public License, v. 2.0.
  */
-.bz_query_head {
+
+#groups_edit_multiple th, .bz_query_head {
   text-align: center;
 }
 
@@ -121,6 +122,15 @@ td.bz_total {
     color: inherit;
 }
 
+.buglist_menu {
+    margin-top: 1em;
+}
+
+.buglist_menu div {
+    display: inline-block;
+    vertical-align: middle;
+}
+
 #groups_edit_multiple, #quips {
     border: 1px solid black;
     border-collapse: collapse;
@@ -132,11 +142,6 @@ td.bz_total {
     padding: 0.3em;
 }
 
-#groups_edit_multiple th, #quips th {
-    text-align: center;
-    white-space: normal !important;
-}
-
 #groups_edit_multiple th.narrow_column {
     width: 8em;
 }
diff --git a/skins/standard/global.css b/skins/standard/global.css
index cf4fe57bf..05a111428 100644
--- a/skins/standard/global.css
+++ b/skins/standard/global.css
@@ -25,6 +25,11 @@
         clear: both;
     }
 
+    #bugzilla-body th {
+        font-weight: bold;
+        vertical-align: top;
+    }
+
 /* global (end) */
 
 /* header (begin) */
@@ -402,6 +407,10 @@ table#flags td {
     padding: 0.3em;
 }
 
+#email_prefs th.role_header {
+    width: 10%;
+}
+
 #bug_activity td {
     vertical-align: top;
 }
@@ -410,6 +419,25 @@ table#flags td {
     padding: 0.5em;
 }
 
+#permissions {
+    margin-left: auto;
+    margin-right: auto;
+    width: 40em;
+}
+
+#permissions p {
+    margin-top: 2em;
+    margin-bottom: 0.5em;
+}
+
+#permissions li {
+    list-style-type: none;
+}
+
+#permissions table td {
+    vertical-align: top;
+}
+
 .column_header {
     background-color: #66f;
 }
@@ -684,6 +712,10 @@ form th {
     text-align: right;
 }
 
+.middle {
+    vertical-align: middle;
+}
+
 .inline {
     display: inline;
 }
diff --git a/skins/standard/params.css b/skins/standard/params.css
index 428ace5ad..491a8a47a 100644
--- a/skins/standard/params.css
+++ b/skins/standard/params.css
@@ -33,13 +33,17 @@
   padding: 0.2em 0.5em;
 }
 
+#params_index {
+  padding-left: 1em;
+}
+
 div.table {
-    display: inline-table;
+  display: inline-table;
 }
 
 div.cell {
-    display: table-cell;
-    vertical-align: top;
+  display: table-cell;
+  vertical-align: top;
 }
 
 td.selected_section {
diff --git a/skins/standard/reports.css b/skins/standard/reports.css
index 7679e32fc..904b8ed26 100644
--- a/skins/standard/reports.css
+++ b/skins/standard/reports.css
@@ -66,13 +66,20 @@
   background-color: #cfffdf; /* light green */
 }
 
-#tabular_report {
+.tabular_report_container {
+  /* Force tables to be horizontally centered */
+  margin-left: auto;
+  margin-right: auto;
+  text-align: center;
+}
+
+.tabular_report {
   border: 1px dotted black;
   border-collapse: collapse;
   text-align: center;
 }
 
-#tabular_report th, #tabular_report td {
+.tabular_report th, .tabular_report td {
   border: 1px dotted;
   padding: 0.2em;
 }
diff --git a/skins/standard/search_form.css b/skins/standard/search_form.css
index 0381730af..c4570bc31 100644
--- a/skins/standard/search_form.css
+++ b/skins/standard/search_form.css
@@ -192,3 +192,34 @@ div.bz_section_title span {
 .bz_simple_search_form th {
   text-align: right;
 }
+
+#tabular_reports_menu, #graphical_reports_menu {
+    width: 30em;
+    margin: 2em auto;
+}
+
+#tabular_reports_menu th, #graphical_reports_menu th {
+    vertical-align: middle;
+    white-space: nowrap;
+    padding: 0.3em;
+}
+
+#tabular_reports_menu th {
+    text-align: center;
+}
+
+#graphical_reports_menu th {
+    text-align: left;
+}
+
+#graphical_reports_menu label {
+    font-weight: normal;
+}
+
+th#reports_menu_multiple_tables {
+    text-align: center;
+    width: 15em;
+    height: 15em;
+    border: 1px solid black;
+    border-collapse: collapse;
+}
diff --git a/skins/standard/summarize-time.css b/skins/standard/summarize-time.css
index 37b135b45..84a27c736 100644
--- a/skins/standard/summarize-time.css
+++ b/skins/standard/summarize-time.css
@@ -6,8 +6,6 @@
  * defined by the Mozilla Public License, v. 2.0.
  */
 
-td { vertical-align: top }
-
 table.zeroitems, table.realitems {
     margin-left: 2.0em;
     margin-top: 2px;
@@ -22,6 +20,9 @@ tr.section_total {
 
 td.subtotal {
     background: #B0C0D9;
+    text-align: right;
+    font-weight: bold;
+    width: 8em;
 }
 
 .zeroitems .bug_header { background: #d0e0f0 }
@@ -51,3 +52,14 @@ table.owner, table.number, table.zeroitems {
     margin: 1em;
     padding: 0.5em;
 }
+
+th.bug_id, th.bug_status {
+    white-space: nowrap;
+    text-align: left;
+    padding-right: 2em;
+    width: 6em;
+}
+
+.time_info {
+    padding-right: 3em;
+}