<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/102611/">http://git.reviewboard.kde.org/r/102611/</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;">Hm, I'm afraid I implemented the same thing some days ago and already pushed it, not being aware of this review request. Sorry about that :-/
>From a quick look over the code your version seems to take care of not writing a default ffffffff color value. That might be something to add to the code in master.
The one I did takes care of writing the original hotspot values without doing any conversion, which I think is cleaner. Do you spot any other differences?
</pre>
<br />
<p>- Dennis</p>
<br />
<p>On September 14th, 2011, 11:27 a.m., Javier Becerra Elcinto 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 Javier Becerra Elcinto.</div>
<p style="color: grey;"><i>Updated Sept. 14, 2011, 11:27 a.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;">Created missing KmlIconStyleTagWriter.cpp and KmlIconStyleTagWriter.h
Added a convenience function GeoDataIconStyle::iconPath() required by KmlIconStyleTagWriter.
When a placemark is created with a QPixmap instead of a filename, the icon is ignored when writing to file (KML does not seem to support icons embedded in kml files, it only allows for hrefs to be used to define them, maybe there is a way to include the binary image somewhere in the document and reference it from the href in the Icon tag?).
</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/geodata/data/GeoDataIconStyle.h <span style="color: grey">(87ec5e1)</span></li>
<li>src/lib/geodata/data/GeoDataIconStyle.cpp <span style="color: grey">(15a2597)</span></li>
<li>src/lib/geodata/writers/kml/KmlIconStyleTagWriter.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/lib/geodata/writers/kml/KmlIconStyleTagWriter.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102611/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>