<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/103146/">http://git.reviewboard.kde.org/r/103146/</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;">Thanks for the patch, works great here with the public transport map! [1]
As usual, here are some minor nitpicks: ;-)
In order to let users zoom in and out, I suggest to activate the Navigation plugin. Moreover, you could help the user judge about the map if you could zoom out such that the whole map becomes visible. For now, it is IMO sufficient to always display the whole planet. As part of a future patch, the area covered by the map should be respected (which seems to be available for the WMS case only).
[1] http://tile.xn--pnvkarte-m4a.de/tilegen/{zoomLevel}/{x}/{y}.png</pre>
<br />
<p>- Bernhard</p>
<br />
<p>On November 15th, 2011, 5:02 p.m., Utku Aydın 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 Utku Aydın.</div>
<p style="color: grey;"><i>Updated Nov. 15, 2011, 5:02 p.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;">Added preview map feature for Map Wizard.</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 with a newly created map from a bitmap image.</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/MapWizard.h <span style="color: grey">(50776bf)</span></li>
<li>src/lib/MapWizard.cpp <span style="color: grey">(9ebd686)</span></li>
<li>src/lib/MapWizard.ui <span style="color: grey">(880b86c)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103146/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>