Fixing cross browser compatability issues on the stepdefinition report

xerxesb · Apr 29, 2010 · 0641546 · 0641546
1 parent 65a3865
commit 0641546
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/Reporting/Common/Common.xslt b/Reporting/Common/Common.xslt
@@ -201,17 +201,18 @@
       <xsl:with-param name="text-key" select="'Hide'" />
     </xsl:call-template>]";
       <![CDATA[
-          function toggle(sdid){
-            e=window.event.srcElement;
+          function toggle(sdid, event){
+            var link = event.target || event.srcElement;
+
             toToggle=document.getElementById(sdid);
-            if (e.innerText==showButtonText)
+            if (link.innerHTML==showButtonText)
             {
-              e.innerText=hideButtonText;
+              link.innerHTML=hideButtonText;
               toToggle.style.display="block";
             }
             else
             {
-              e.innerText=showButtonText;
+              link.innerHTML=showButtonText;
               toToggle.style.display="none";
             }
           }

diff --git a/Reporting/StepDefinitionReport/StepDefinitionReport.xslt b/Reporting/StepDefinitionReport/StepDefinitionReport.xslt
@@ -160,7 +160,7 @@
               <xsl:value-of select="count(./sfr:Instances/sfr:Instance)"/>
               <xsl:if test="sfr:Instances">
                 <xsl:text> </xsl:text>
-                <a href="#" onclick="toggle('{$stepdef-id}'); return false;" class="button">[<xsl:call-template name="get-common-tool-text">
+                <a href="#" onclick="toggle('{$stepdef-id}', event); return false;" class="button">[<xsl:call-template name="get-common-tool-text">
                   <xsl:with-param name="text-key" select="'Show'" />
                 </xsl:call-template>]</a>
               </xsl:if>