Bug 952795: Remove all style="..." attributes from HTML elements

r/a=justdave

6 files changed, 110 insertions, 10 deletions

diff --git a/skins/standard/admin.css b/skins/standard/admin.css
index 7058bd1af..d0aaf368f 100644
--- a/skins/standard/admin.css
+++ b/skins/standard/admin.css
@@ -43,6 +43,20 @@ p.areyoureallyreallysure {
     font-weight: bold;
 }
 
+#admin_table {
+    border-collapse: collapse;
+    border: 1px solid black;
+}
+
+#admin_table th {
+    white-space: normal !important;
+}
+
+#admin_table th, #admin_table td {
+    border: 1px solid black;
+    padding: 0.3em;
+}
+
 tr.disabled {
     background-color: lightgrey;
 }
@@ -140,3 +154,29 @@ th.title {
 #flag_types_bugs .multiplicable, #flag_types_attachments .multiplicable {
     display: block;
 }
+
+table.whining_list {
+    border: 1px solid black;
+    border-collapse: collapse;
+    width: 100%;
+}
+
+table.whining_list th, table.whining_list td {
+    padding: 0.3em;
+}
+
+table.whining_list th {
+    text-align: right;
+    font-weight: normal !important;
+    white-space: normal !important;
+}
+
+table.whining_list th.subtitle {
+    font-weight: bold !important;
+    text-align: left !important;
+}
+
+table.schedule_list th, table.search_list th {
+    text-align: center;
+    font-weight: bold !important;
+}
diff --git a/skins/standard/dependency-tree.css b/skins/standard/dependency-tree.css
index 0051fca36..efc2e8134 100644
--- a/skins/standard/dependency-tree.css
+++ b/skins/standard/dependency-tree.css
@@ -78,3 +78,7 @@ ul.tree a.tree_link img {
     color: #333;
     font-weight: bold;
 }
+
+.dependency_tree_controls input[type=submit] {
+    min-width: 3em;
+}
diff --git a/skins/standard/editusers.css b/skins/standard/editusers.css
index fb38af67d..05cee4b39 100644
--- a/skins/standard/editusers.css
+++ b/skins/standard/editusers.css
@@ -34,12 +34,12 @@ table.groups tr.indirect {
 }
 table.groups th {
     text-align: left;
-    padding: 0 0 0 1ex;
+    padding: 0 1ex;
 }
 table.groups td {
     padding: 2px;
 }
-table.groups td.checkbox {
+table.groups th.checkbox, table.groups td.checkbox {
     text-align: center;
     white-space: nowrap;
 }
diff --git a/skins/standard/global.css b/skins/standard/global.css
index 9396a6346..ecf39c6a8 100644
--- a/skins/standard/global.css
+++ b/skins/standard/global.css
@@ -397,10 +397,6 @@ dl dl > dt {
     border-top: none;
 }
 
-#admin_table th {
-    white-space: normal !important;
-}
-
 /* Arrow buttons are buttons with only ↑, ↓, ← or → on
  * them. We want these to look a little less spidery. */
 .arrow_button {
@@ -427,7 +423,7 @@ dl dl > dt {
     font-size: smaller;
 }
 
-.bz_attach_flags, .bz_attach_footer {
+.bz_attach_flags, .bz_attach_footer, .bz_flags, .nowrap {
     white-space: nowrap;
 }
 
@@ -440,6 +436,10 @@ div.user_match {
     margin-bottom: 1em;
 }
 
+.indented {
+    margin-left: 3em;
+}
+
 .box {
     border: 1px solid black;
     color: black;
@@ -452,6 +452,12 @@ div.user_match {
     display: none;
 }
 
+#serverpush_msg {
+    margin-top: 20%;
+    text-align: center;
+    font-size: xx-large;
+}
+
 /* Rules specific for printing */
 @media print {
     #header, #footer {
@@ -480,9 +486,14 @@ div.user_match {
 .field_help_link {
     cursor: help;
 }
-.field_value, form#Create th, form#Create td {
+.field_value, form#Create th, form#Create td, .top {
     vertical-align: top;
 }
+
+.bottom {
+    vertical-align: bottom;
+}
+
 .field_value .text_input {
   width: 100%;
   min-width: 25em;
@@ -578,10 +589,23 @@ input.required, select.required, span.required_explanation {
 form th {
     text-align: right;
 }
-th.left {
+
+.left {
     text-align: left;
 }
 
+.center {
+    text-align: center;
+}
+
+.right {
+    text-align: right;
+}
+
+.inline {
+    display: inline;
+}
+
 tr.shared_search {
     background-color: #fff7cd;
     color: #000;
diff --git a/skins/standard/reports.css b/skins/standard/reports.css
index 8e0ddf1b0..4db7d2dac 100644
--- a/skins/standard/reports.css
+++ b/skins/standard/reports.css
@@ -106,4 +106,30 @@
 .extra_info {
   font-size: smaller;
   font-style: italic;
-}
\ No newline at end of file
+}
+
+#edit_series th, #create_chart th, #edit_charts th {
+    text-align: center;
+}
+
+#edit_series th, #edit_series td,
+#create_chart th, #create_chart td {
+    vertical-align: top;
+    padding: 0.5em;
+}
+
+#edit_series select, #create_chart select {
+    width: 15em;
+}
+
+#edit_series input {
+    width: 95%;
+}
+
+#edit_series input#frequency {
+    width: 3em;
+}
+
+#edit_charts input#action-sum, #edit_charts input#action-remove {
+    width: 7em;
+}
diff --git a/skins/standard/summarize-time.css b/skins/standard/summarize-time.css
index bb42da6fe..c7ed96827 100644
--- a/skins/standard/summarize-time.css
+++ b/skins/standard/summarize-time.css
@@ -35,3 +35,9 @@ td.subtotal {
 /* the details headers */
 .number .owner_header, .owner .bug_header { background: #ffffff }
 .number .owner_header2, .owner .bug_header2 { background: #EFEFEF }
+
+.warning_swapped_data {
+    border: 1px solid red;
+    margin: 1em;
+    padding: 0.5em;
+}