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

r/a=justdave

3 files changed, 20 insertions, 27 deletions

diff --git a/template/en/default/bug/create/create-guided.html.tmpl b/template/en/default/bug/create/create-guided.html.tmpl
index 181b76e62..7abb8c7dc 100644
--- a/template/en/default/bug/create/create-guided.html.tmpl
+++ b/template/en/default/bug/create/create-guided.html.tmpl
@@ -62,7 +62,6 @@ function PutDescription() {
 </p>
 
 <iframe name="somebugs" id="somebugs"
-  style="border: 2px black solid"
   src="duplicates.cgi?[% productstring %]&amp;format=simple">
 </iframe>
 
diff --git a/template/en/default/bug/dependency-tree.html.tmpl b/template/en/default/bug/dependency-tree.html.tmpl
index 369326d48..61994cf28 100644
--- a/template/en/default/bug/dependency-tree.html.tmpl
+++ b/template/en/default/bug/dependency-tree.html.tmpl
@@ -166,9 +166,8 @@
    <tr>
    [%# Hide/show resolved button
        Swaps text depending on the state of hide_resolved %]
-   <td align="center">
-   <form method="get" action="showdependencytree.cgi"
-           style="display: inline; margin: 0px;">
+   <td>
+   <form method="get" action="showdependencytree.cgi">
      <input name="id" type="hidden" value="[% bugid %]">
      [% IF maxdepth %]
        <input name="maxdepth" type="hidden" value="[% maxdepth %]">
@@ -188,11 +187,9 @@
    </td>
 
    <td>
-   <form method="get" action="showdependencytree.cgi"
-         style="display: inline; margin: 0px;">
+   <form method="get" action="showdependencytree.cgi">
      [%# set to one form %]
-     <input type="submit" id="change_maxdepth"
-       value="&nbsp;1&nbsp;" 
+     <input type="submit" id="change_maxdepth" value="1"
        [% "disabled" IF realdepth < 2 || maxdepth == 1 %]>
      <input name="id" type="hidden" value="[% bugid %]">
      <input name="maxdepth" type="hidden" value="1">
@@ -201,8 +198,7 @@
    </td>
 
    <td>
-   <form method="get" action="showdependencytree.cgi"
-         style="display: inline; margin: 0px;">
+   <form method="get" action="showdependencytree.cgi">
      [%# Minus one form
          Allow subtracting only when realdepth and maxdepth > 1 %]
      <input name="id" type="hidden" value="[% bugid %]">
@@ -211,14 +207,13 @@
                        : ( maxdepth ? maxdepth - 1 : realdepth - 1 )
      %]">
      <input name="hide_resolved" type="hidden" value="[% hide_resolved %]">
-     <input type="submit" id="decrease_depth" value="&nbsp;&lt;&nbsp;"
+     <input type="submit" id="decrease_depth" value="&lt;"
        [% "disabled" IF realdepth < 2 || ( maxdepth && maxdepth < 2 ) %]>
    </form>
    </td>
 
    <td>
-   <form method="get" action="showdependencytree.cgi"
-         style="display: inline; margin: 0px;">
+   <form method="get" action="showdependencytree.cgi">
      [%# Limit entry form: the button cannot do anything when total depth
          is less than two, so disable it %]
      <input name="maxdepth" size="4" maxlength="4" value="[%
@@ -234,8 +229,7 @@
    </td>
 
    <td>
-   <form method="get" action="showdependencytree.cgi"
-         style="display: inline; margin: 0px;">
+   <form method="get" action="showdependencytree.cgi">
      [%# plus one form
          Disable button if total depth < 2, or if depth set to unlimited %]
      <input name="id" type="hidden" value="[% bugid %]">
@@ -243,19 +237,18 @@
        <input name="maxdepth" type="hidden" value="[% maxdepth + 1 %]">
      [% END %]
      <input name="hide_resolved" type="hidden" value="[% hide_resolved %]">
-     <input type="submit" id="increase_depth" value="&nbsp;&gt;&nbsp;" 
+     <input type="submit" id="increase_depth" value="&gt;"
         [% "disabled" IF realdepth < 2 || !maxdepth || maxdepth >= realdepth %]>
    </form>
    </td>
 
    <td>
-   <form method="get" action="showdependencytree.cgi"
-         style="display: inline; margin: 0px;">
+   <form method="get" action="showdependencytree.cgi">
      [%# Unlimited button %]
      <input name="id" type="hidden" value="[% bugid %]">
      <input name="hide_resolved" type="hidden" value="[% hide_resolved %]">
      <input type="submit" id="remove_limit"
-       value="&nbsp;Unlimited&nbsp;"
+       value="Unlimited"
        [% "disabled" IF maxdepth == 0 || maxdepth == realdepth %]>
    </form>
    </td>
diff --git a/template/en/default/bug/summarize-time.html.tmpl b/template/en/default/bug/summarize-time.html.tmpl
index 932a6e3f4..ee654dc38 100644
--- a/template/en/default/bug/summarize-time.html.tmpl
+++ b/template/en/default/bug/summarize-time.html.tmpl
@@ -36,7 +36,7 @@
     [%# parts contains date ranges (from, to). %]
     [% part = parts.shift %]
     <div align="right">
-      <h4 style="padding-right: 2em; margin: 0;">
+      <h4>
         [% IF part.0 or part.1 %]
           [% part.0 OR "Up" FILTER html %] to [% part.1 OR "now" FILTER html %]
         [% ELSE %]
@@ -52,7 +52,7 @@
   [% END %]
 
   [% IF detailed %]
-    <h4 style="margin: 0">
+    <h4>
       Total of [% global.remaining FILTER format("%.2f") %]h remains from
       original estimate of [% global.estimated FILTER format("%.2f") %]h
       [% IF global.deadline %]
@@ -62,13 +62,13 @@
   [% END %]
 
   [% IF monthly %]
-    <h4 style="margin: 0">Total of [% global.grand_total FILTER format("%.2f") %] hours worked</h4>
+    <h4>Total of [% global.grand_total FILTER format("%.2f") %] hours worked</h4>
     <hr noshade size="1">
   [% END %]
 
   [% IF null.size > 0 %]
     [% INCLUDE inactive_report %]
-    <h4 style="margin: 0">Total of [% null.size %] inactive [% terms.bugs %]</h4>
+    <h4>Total of [% null.size %] inactive [% terms.bugs %]</h4>
   [% END %]
 
 [% END %]
@@ -255,15 +255,16 @@
 
 [% BLOCK query_form %]
 <hr noshade size=1>
-<form method="post" name="summary" style="display: inline" action="summarize_time.cgi">
+<form method="post" name="summary" action="summarize_time.cgi">
 <input type="hidden" name="do_depends" value="[% do_depends FILTER html %]">
 <input type="hidden" name="id" value="[% ids.join(",") FILTER html %]">
 <input type="hidden" name="do_report" value="1">
 
 [% IF warn_swap_dates %]
-    <h4 style="border: 1px solid red; margin: 1em; padding: 0.5em">The
-    end date specified occurs before the start date, which doesn't
-    make sense; the dates below have therefore been swapped.</h4>
+  <p class="warning_swapped_data">
+    The end date specified occurs before the start date, which doesn't
+    make sense; the dates below have therefore been swapped.
+  </p>
 [% END %]
 
 <table>