<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="https://git.reviewboard.kde.org/r/118495/">https://git.reviewboard.kde.org/r/118495/</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;">Looks good. There's a regression here (running Qt 4.8) regarding the overview map and the scale bar, they do not receive (or at least handle) mouse events anymore. Therefore moving or hiding these two is not possible, for example. Other ones seem to work fine though.
</pre>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/118495/diff/1/?file=278247#file278247line73" style="color: black; font-weight: bold; text-decoration: underline;">examples/cpp/marbleQuick2/main.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">72</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">if</span> <span class="p">(</span><span class="n">scale</span> <span class="o"><</span> <span class="mf">0.5</span><span class="p">)</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">scale = qBound(0.5, scale, 2.0);</pre>
</div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/118495/diff/1/?file=278252#file278252line18" style="color: black; font-weight: bold; text-decoration: underline;">src/lib/marble/MarbleInputHandler.h</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">18</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">#include <QRect>
missing for Qt 4.8
</pre>
</div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/118495/diff/1/?file=278257#file278257line21" style="color: black; font-weight: bold; text-decoration: underline;">src/lib/marble/MarbleWidgetInputHandler.h</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">20</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#include <QString></span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">17</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#include "MarbleInputHandler.h"</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">#include <QSharedPointer>
missing for Qt 4.8</pre>
</div>
<br />
<p>- Dennis Nienhüser</p>
<br />
<p>On June 3rd, 2014, 12:36 p.m. UTC, Adam Dabrowski wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Marble and Dennis Nienhüser.</div>
<div>By Adam Dabrowski.</div>
<p style="color: grey;"><i>Updated June 3, 2014, 12:36 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
marble
</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;">Functionalities:
- Mouse, touch and pinch input handling for MarbleQuickItem (keyboard was enabled earlier)
Other changes:
- Refactored input handling to reuse code, broke into smaller pieces (functions)
- Extended the example (marbleQuick2)
Known issues:
- Some refactoring still should be done around input classes
- Inertial rotation is disabled by default in MarbleQuickItem because of some issues (mostly touch-related). However, issues are rather minor so feel free to enable it back.
</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;">- Using these changes since some time in a working app that is based on a MarbleQuickItem subclass (on both unix PC and a touch device)
- Built and run marbleQuick2 example on both PC and a touch device.
- Did good tests for MarbleQuickItem, basic regression testing for MarbleWidget and no testing for Qt4.</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>examples/cpp/marbleQuick2/main.cpp <span style="color: grey">(640baa6)</span></li>
<li>examples/cpp/marbleQuick2/main.qml <span style="color: grey">(76947ed)</span></li>
<li>src/lib/marble/CMakeLists.txt <span style="color: grey">(4add26b)</span></li>
<li>src/lib/marble/MarbleAbstractPresenter.h <span style="color: grey">(7c047ed)</span></li>
<li>src/lib/marble/MarbleAbstractPresenter.cpp <span style="color: grey">(98855bb)</span></li>
<li>src/lib/marble/MarbleInputHandler.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/lib/marble/MarbleInputHandler.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/lib/marble/MarbleQuickItem.h <span style="color: grey">(8fd5657)</span></li>
<li>src/lib/marble/MarbleQuickItem.cpp <span style="color: grey">(0ccc0ac)</span></li>
<li>src/lib/marble/MarbleWidget.cpp <span style="color: grey">(acc270a)</span></li>
<li>src/lib/marble/MarbleWidgetInputHandler.h <span style="color: grey">(f763d1e)</span></li>
<li>src/lib/marble/MarbleWidgetInputHandler.cpp <span style="color: grey">(912c12d)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118495/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>