<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/102571/">http://git.reviewboard.kde.org/r/102571/</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;">Great work! I've been wanting a speedometer for a while, acutally. I'm wondering, though, if this is the main use case of your speed plugin?
Personally, my main use case would be to track current speed and distance. Therefore, I'd like to see the speed value in big fat letters. The distance should be visible somewhere as well, but probably not as much exposed as the speed. Showing the direction and presicion, however, seem rather redundant to me, because these values are already visualized by the position marker plugin. The altitude would be nice to see as well, though I could imagine that it is more appealing if it was rendered as a grap. Perhaps this should even be a separate plugin.
What do you think?</pre>
<br />
<p>- Bernhard</p>
<br />
<p>On September 10th, 2011, 7:39 p.m., Thibaut Gridel 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 Thibaut Gridel.</div>
<p style="color: grey;"><i>Updated Sept. 10, 2011, 7:39 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;">Provides a small box with speed, direction, altitude and precision.
Useful while checking speed limits...</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/CMakeLists.txt <span style="color: grey">(548b282)</span></li>
<li>src/plugins/render/speed/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugins/render/speed/Speed.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugins/render/speed/Speed.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugins/render/speed/SpeedPlugin.ui <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102571/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/102571/s/253/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/09/10/speed_plugin_400x100.jpeg" style="border: 1px black solid;" alt="Speed Plugin" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>