plain-reload: Constrain tail output to 50 lines.
There are quite a lot of build outputs which have lines with a length exceeding the width of the taillog <pre/> and thus visually produce more lines than 50. This causes the tail "box" to change height frequently and to get to the bottom you need to scroll down. We now set a fixed line-height to 120% of the font size and cap the maximum height based on that value (50 * 1.2 = 60). It's probably not nice to override the line-height, but max-lines is currently only available using browser-specific property names. But after all it's just for the tail output, if people complain about the line-height, we can still change it :-) Signed-off-by: aszlig <aszlig@redmoonstudios.org>
This commit is contained in:
parent
be051bcadc
commit
c251598241
|
@ -15,6 +15,7 @@ function injectTail() {
|
||||||
dataType: "text",
|
dataType: "text",
|
||||||
success: function (tail) {
|
success: function (tail) {
|
||||||
$("#contents").text(tail);
|
$("#contents").text(tail);
|
||||||
|
$("#contents").scrollTop($("#contents").get(0).scrollHeight);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
|
@ -119,3 +119,9 @@ span.keep-whitespace {
|
||||||
.build-status {
|
.build-status {
|
||||||
max-width: none; /* don't apply responsive design to status images */
|
max-width: none; /* don't apply responsive design to status images */
|
||||||
}
|
}
|
||||||
|
|
||||||
|
pre.taillog {
|
||||||
|
line-height: 1.2em;
|
||||||
|
max-height: 60em;
|
||||||
|
overflow: hidden;
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in a new issue