[Marble-devel] Review Request: Gps Speed render plugin

[Bernhard Beschow]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102571/#review6423
-----------------------------------------------------------


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?

- Bernhard


On Sept. 10, 2011, 7:39 p.m., Thibaut Gridel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102571/
> -----------------------------------------------------------
> 
> (Updated Sept. 10, 2011, 7:39 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Summary
> -------
> 
> Provides a small box with speed, direction, altitude and precision.
> 
> Useful while checking speed limits...
> 
> 
> Diffs
> -----
> 
>   src/plugins/render/CMakeLists.txt 548b282 
>   src/plugins/render/speed/CMakeLists.txt PRE-CREATION 
>   src/plugins/render/speed/Speed.h PRE-CREATION 
>   src/plugins/render/speed/Speed.cpp PRE-CREATION 
>   src/plugins/render/speed/SpeedPlugin.ui PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/102571/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> Speed Plugin
>   http://git.reviewboard.kde.org/r/102571/s/253/
> 
> 
> Thanks,
> 
> Thibaut
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20110911/1e1ddf44/attachment.html>