<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/101506/">http://git.reviewboard.kde.org/r/101506/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 5914f88aa71b73495e38ee60db1674290db5743f by Aurélien Gâteau to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On June 4th, 2011, 3:10 p.m., Aurélien Gâteau wrote:</p>
<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 KDevelop.</div>
<div>By Aurélien Gâteau.</div>
<p style="color: grey;"><i>Updated June 4, 2011, 3:10 p.m.</i></p>
<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;">When KDevelop window is maximized, the buttons on the left and right edges of the window could be easier to reach. Right now there is a 2 pixel gap between the button and the border, which makes it impossible to reach the button by throwing the mouse to the edge.
Attached simple patch just set the margin to 0. I am not sure however if it makes sense to keep the define if it is set to 0. If you prefer me to get rid of the define and replace its usage with plain 0, tell me.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">See attached before/after screenshot.</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>sublime/ideallayout.h <span style="color: grey">(06adbe7)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101506/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/101506/s/174/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/06/04/kdevplatform-no-margin_400x100.png" style="border: 1px black solid;" alt="Before - After" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>