<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/119317/">https://git.reviewboard.kde.org/r/119317/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Marble, Dennis Nienhüser and Torsten Rahn.</div>
<div>By Cruceru Calin-Cristian.</div>
<p style="color: grey;"><i>Updated July 16, 2014, 4:23 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 (updated)</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch includes 2 changes:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- Annotate Plugin actions menu redesigned:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
* The action group is now exclusive, so no more checking and no more actions selected at the same time;<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
* I removed the 'Enable Moving Map' option since it does not have any purpose yet. Also, from some reasons, it was not working properly: unchecking and checking it again caused both annotate items and the map to catch the events.<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
* I added a new action, 'Select item' which corresponds to the 'Editing' state - when having this action selected, the user can move the nodes/polygon, remove nodes, pop-up de Properties menu, etc (somehow it is the equivalent of nothing being checked before making this group actions exclusive).<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- Added node highlighting on hovering events (mouse move events) in both 'Editing' state and 'MergingNodesState'. Also, the cursor changes according to these hovering events (please tell me if other cursors should be used - I used the two ones you will see just for testing and they seemed to me the most appropriate).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">PS: Testing with marble-qt provides a nicer interface owing to the toolbar buttons. I recommend using marble-qt since this interface is closer to how the final UI will be like than the KDE version (where the actions are only within the View menu).</p></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/plugins/render/annotate/AnnotatePlugin.h <span style="color: grey">(6f806c4)</span></li>
<li>src/plugins/render/annotate/AnnotatePlugin.cpp <span style="color: grey">(6129b1a)</span></li>
<li>src/plugins/render/annotate/AreaAnnotation.h <span style="color: grey">(f4f319a)</span></li>
<li>src/plugins/render/annotate/AreaAnnotation.cpp <span style="color: grey">(b49d678)</span></li>
<li>src/plugins/render/annotate/GroundOverlayFrame.cpp <span style="color: grey">(68fda48)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119317/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>