<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/119537/">https://git.reviewboard.kde.org/r/119537/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="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;">Ship It!</p></pre>
<br />
<p>- Torsten Rahn</p>
<br />
<p>On Juli 29th, 2014, 2:48 nachm. UTC, Cruceru Calin-Cristian wrote:</p>
<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.</div>
<div>By Cruceru Calin-Cristian.</div>
<p style="color: grey;"><i>Updated Juli 29, 2014, 2:48 nachm.</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I removed the bubble which popped up when clicking placemarks and I added a new way to customize placemarks - an edit dialog. This has customizing options for the following:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- link to the icon;<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- placemark's coordinates;<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- description;<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- label color/scale;<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- icon color/scale.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The way it works is the following: just click the 'Add Placemark' action in the View menu (or in the toolbar if you are running marble-qt) and a placemark will automatically be added on the focus point on the map and the dialog will show up. Then, if the 'Close' button is clicked, the placemark which has just been added gets removed (it most probably means that the 'Add placemark' button has been pressed by mistake). If the 'Ok' button is pressed, the placemark gets saved. What is more, any change on dialog's fields gets reflected on placemark's style/position immediately. I found this behaviour nicer than the 'Apply' method which has been used on Polygon's edit dialog and I'm planning to change that to be consistent with this one. If you are ok with this, of course.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">There are, still, a couple of problems, which will take some time to deal with, but are not very imperious. These are:<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- the 'scale' field on both icon and label does not work at the moment. idis told me that most probably the scale is not considered when rendering the placemarks. I found a way to make it 'change something' (for label so far), but I'm not sure if this is what should happen, it label becomes very low-quality. See the attached patch. Please apply it after applying the main patch and tell me what you think about it. What else should be modified so that the label's text would become larger, not just zoomed (how it looks with this patch).<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
- the color selector field for icon does nothing for the moment, simply because I'm using GeoDataColorStyle::setColor() which when being called from a GeoDataIconStyle instance does not actually affect the icon (the image) - which in fact is obvious. There are a couple of options here. One would be to create a couple of placemark icons, save them as .svg and implement a mechanism to change their color (using QGraphicsSvgItem for example).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'm waiting feedback on the feature this patch adds as well as suggestions on the issues I listed above.</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/EditGroundOverlayDialog.ui <span style="color: grey">(8c88476)</span></li>
<li>src/plugins/render/annotate/EditTextAnnotationDialog.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugins/render/annotate/EditTextAnnotationDialog.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugins/render/annotate/EditTextAnnotationDialog.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugins/render/annotate/PlacemarkTextAnnotation.h <span style="color: grey">(733664b)</span></li>
<li>src/plugins/render/annotate/PlacemarkTextAnnotation.cpp <span style="color: grey">(d56b8a6)</span></li>
<li>src/plugins/render/annotate/SceneGraphicsItem.h <span style="color: grey">(d9d5bfe)</span></li>
<li>src/plugins/render/annotate/CMakeLists.txt <span style="color: grey">(b23ae06)</span></li>
<li>src/plugins/render/annotate/AnnotatePlugin.h <span style="color: grey">(26e8483)</span></li>
<li>src/plugins/render/annotate/AnnotatePlugin.cpp <span style="color: grey">(7e4b92f)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119537/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/07/29/bf82af3f-4b56-4f0b-8ab4-be680271aea9__Applying-scale.diff">Applying scale</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>