<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/101417/">http://git.reviewboard.kde.org/r/101417/</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 1ade66705b1c9f3484383236f1c8757432d51591 by Marco Martin.</pre>
<br />
<p>- Commit</p>
<br />
<p>On May 22nd, 2011, 2:43 p.m., Marco Martin 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 Marble.</div>
<div>By Marco Martin.</div>
<p style="color: grey;"><i>Updated May 22, 2011, 2:43 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;">at start, the multitouch pinch zoom in marble was implemented with raw qtouchevent because the gesture framework still wasn't available.
this patch changes marble to use the factory pinch gesture, that simplifies a bit the code and should give a more coherent behaviour compared to other apps that use the same stock gesture.</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;">tested on meego on a lenovo ideapad. the amount of zooming now seems matematically more correct after the gesture.
with both old and new code there is an ugly delay during zooming, that is probably due to a misbehaviour in the qt multitouch itself (either too much or not enough events arriving), it's probably not depended from marble itself, it should be tried either on mac or windows to see if the problem is present also where multitouch is better supported in Qt</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/lib/MarbleWidget.cpp <span style="color: grey">(39eb47a)</span></li>
<li>src/lib/MarbleWidgetInputHandler.cpp <span style="color: grey">(2a31a8b)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101417/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>