<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 />
<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>
<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>