<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/105124/">http://git.reviewboard.kde.org/r/105124/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for rekonq and Andrea Diamantini.</div>
<div>By Anton Kreuzkamp.</div>
<p style="color: grey;"><i>Updated June 2, 2012, 8:40 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I added a screenshot with the same image on a grey background. (I'll update the patch, if you like it)
It somehow reminds me of thunderclouds. This dark, depressive-looking grey with the schemes in it. It has something very emotional. I like it. Not sure whether this fits here, though.
Alternatively we could stick with the totally grey background. It looks least fancy, but most functional and probably most like "error".
PS: I'd like to know whether you prefer the bright, or the Plasma-like boxes, as I think it's better to have only one kind of boxes for the newtab- and error-pages.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Add a background-image, a box-shadow and a warning-symbol to indicate it's a error-page. See screenshot.
I've created an alternative version, with a more Plasma-like box, I've posted it in a similar review-request for the new tab page (https://git.reviewboard.kde.org/r/105123/).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/data/rekonqinfo.html <span style="color: grey">(35fade4)</span></li>
<li>src/webpage.cpp <span style="color: grey">(bc4d0aa)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105124/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots (updated)</h1>
<div>
<a href="http://git.reviewboard.kde.org/r/105124/s/586/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/05/31/rekonq-error-page_400x100.png" style="border: 1px black solid;" alt="The error page with this patch" /></a>
<a href="http://git.reviewboard.kde.org/r/105124/s/588/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/06/02/rekonq-errorpage-grey_400x100.png" style="border: 1px black solid;" alt="A grey alternative (not in this patch))" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>