Fix for bug 98021: Cleans up "edit attachment" interface on NS4.x by removing text of buttons that do not work in that browser.

Patch by Myk Melez <myk@mozilla.org>.
r=kiko x 2

1 files changed, 11 insertions, 3 deletions

diff --git a/template/default/attachment/edit.atml b/template/default/attachment/edit.atml
index 49984dc00..61452145a 100755
--- a/template/default/attachment/edit.atml
+++ b/template/default/attachment/edit.atml
@@ -80,9 +80,15 @@
             <b>You cannot view the attachment while editing it because your browser does not support IFRAMEs.
             <a href="attachment.cgi?id=[% attachid %]&action=view">View the attachment on a separate page</a>.</b>
           </iframe>
-          <button id="editButton" onclick="editAsComment();">Edit Attachment As Comment</button>
-          <button id="undoEditButton" onclick="undoEditAsComment();" style="display: none;">Undo Edit As Comment</button>
-          <button id="redoEditButton" onclick="redoEditAsComment();" style="display: none;">Redo Edit As Comment</button>
+          <script type="application/x-javascript" language="JavaScript">
+            <!--
+            if (typeof document.getElementById == "function") {
+              document.write('<button id="editButton" onclick="editAsComment();">Edit Attachment As Comment</button>');
+              document.write('<button id="undoEditButton" onclick="undoEditAsComment();" style="display: none;">Undo Edit As Comment</button>');
+              document.write('<button id="redoEditButton" onclick="redoEditAsComment();" style="display: none;">Redo Edit As Comment</button>');
+            }
+            //-->
+          </script>
         </td>
       [% ELSE %]
         <td id="noview" width="50%">
@@ -110,6 +116,7 @@
 </form>
 
 <script type="application/x-javascript" language="JavaScript">
+  <!--
   function editAsComment()
     {
       // Get the content of the document as a string.
@@ -210,6 +217,7 @@
       big.parentNode.removeChild(big);
     }
   }
+  //-->
 </script>
 
 <br>