Skip to content
Browse files
[JENKINS-20772] Only show error message in dialog
This change removes the superfluous UI elements around the error
message if possible. Otherwise, fall back to existing behavior
of adding entire page to iframe.

(cherry picked from commit b856cc2)
  • Loading branch information
daniel-beck authored and olivergondza committed Jan 21, 2014
1 parent 0a52f19 commit fb738ea6de6bbcc7480705d53995cd8ec88e6f40
Showing 2 changed files with 9 additions and 2 deletions.
@@ -41,6 +41,7 @@ THE SOFTWARE.
<h1 style="text-align: center">
<img src="${imagesURL}/rage.png" height="179" width="154"/> <span style="font-size:50px"><st:nbsp/>${%Oops!}</span>
<div id="error-description">
@@ -51,6 +52,7 @@ THE SOFTWARE.
<h2>${%Stack trace}</h2>
<pre style="margin:2em; clear:both">${h.printThrowable(request.getAttribute('javax.servlet.error.exception'))}</pre>
@@ -12,7 +12,7 @@ Behaviour.specify("INPUT.apply-button", 'apply', 0, function (e) {

responseDialog.setBody("<div id='"+containerId+"'></iframe>");
responseDialog.setBody("<div id='"+containerId+"'></div>");
var target; // iframe

@@ -42,7 +42,12 @@ Behaviour.specify("INPUT.apply-button", 'apply', 0, function (e) {
} else {
// otherwise this is possibly an error from the server, so we need to render the whole content.
var doc = target.contentDocument || target.contentWindow.document;
var error = doc.getElementById('error-description');
if (!error) {
// fallback if it's not a regular error dialog from oops.jelly: use the entire body
error = doc.getElementsByTagName('body')[0];
var r = YAHOO.util.Dom.getClientRegion();

0 comments on commit fb738ea

Please sign in to comment.