<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://reviewboard.kde.org/r/4522/">http://reviewboard.kde.org/r/4522/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre>looks completely sensible to me.</pre>
<br />
<p>- Aaron</p>
<br />
<p>On July 5th, 2010, 6:31 p.m., Beat Wolf wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://reviewboard.kde.orgrb/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 Plasma.</div>
<div>By Beat Wolf.</div>
<p style="color: grey;"><i>Updated 2010-07-05 18:31:20</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;">Recalculate the current desktop size in kephal and don't trust QDesktopwidget on that one.
Solves an issue that the current desktop size is wrong just after the screen added signal has been received.
Solves the bug in pager where the aspect ratio is not changed correctly when adding and removing a screen.
Of course cashing could be done, but since there are usualy only 1-2 screens and that method is not used often, it should be just fine like that.
PS: the withespace thing will be corrected when i commit</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;">Tested with a second screen</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>trunk/KDE/kdebase/workspace/libs/kephal/kephal/screens.cpp <span style="color: grey">(1146308)</span></li>
</ul>
<p><a href="http://reviewboard.kde.org/r/4522/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>