TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Merge pull request #9313 from bkzl/return-false-on-toggle-on-error-pages

Browse Source 
Return false in toggle function on error pages
This commit is contained in:
Guillermo Iguaran 2013-02-17 12:53:21 -08:00
commit b04c81d367
2 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
actionpack/lib/action_dispatch/middleware/templates/rescues/_request_and_response.erb
View File

@ -1,6 +1,6 @@
<% unless @exception.blamed_files.blank? %>
<% if (hide = @exception.blamed_files.length > 8) %>
<a href="#" onclick="toggleTrace()">Toggle blamed files</a>
<a href="#" onclick="return toggleTrace()">Toggle blamed files</a>
<% end %>
<pre id="blame_trace" <%='style="display:none"' if hide %>><code><%= @exception.describe_blame %></code></pre>
<% end %>
@ -21,12 +21,12 @@
<p><b>Parameters</b>:</p> <pre><%= request_dump %></pre>
<div class="details">
<div class="summary"><a href="#" onclick="toggleSessionDump()">Toggle session dump</a></div>
<div class="summary"><a href="#" onclick="return toggleSessionDump()">Toggle session dump</a></div>
<div id="session_dump" style="display:none"><pre><%= debug_hash @request.session %></pre></div>
</div>
<div class="details">
<div class="summary"><a href="#" onclick="toggleEnvDump()">Toggle env dump</a></div>
<div class="summary"><a href="#" onclick="return toggleEnvDump()">Toggle env dump</a></div>
<div id="env_dump" style="display:none"><pre><%= debug_hash @request.env.slice(*@request.class::ENV_METHODS) %></pre></div>
</div>

7
actionpack/lib/action_dispatch/middleware/templates/rescues/layout.erb
View File

@ -121,6 +121,7 @@
var toggle = function(id) {
var s = document.getElementById(id).style;
s.display = s.display == 'none' ? 'block' : 'none';
return false;
}
var show = function(id) {
document.getElementById(id).style.display = 'block';
@ -129,13 +130,13 @@
document.getElementById(id).style.display = 'none';
}
var toggleTrace = function() {
toggle('blame_trace');
return toggle('blame_trace');
}
var toggleSessionDump = function() {
toggle('session_dump');
return toggle('session_dump');
}
var toggleEnvDump = function() {
toggle('env_dump');
return toggle('env_dump');
}
</script>
</head>