<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/102540/">http://git.reviewboard.kde.org/r/102540/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 10th, 2011, 4:16 p.m., <b>Torsten Rahn</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I like this feature in general.
But on one hand there's a bug:
*If I press "Cancel" in the QColorDialog then the color chosen becomes "black". In that case the prior color should be preserved.
And on the other hand I wonder whether it's done in the right place: It seems that there can be only one route that can be changed in terms of colors. So there is only a single color that can be modified. Also changing the color of the route is something that probably isn't done that often.
So in order to avoid clutter in the RMB menu, could we move this feature into the plugin settings dialog? Have a look at the following plugins and their configuration dialogs (under Settings->Plugin page):
* Grid
* Position Marker
* Earthquake
All of them are doing the color setting there. I think it's the appropriate place given the likeliness that the user will have an issue with the color and given the expected amount of times that he will change it. Also you can have a look at the code in those places how the color settings are saved :-)
Florian: Welcome to the Marble Team! - Great to have you on board - I hope you don't feel discouraged by my comments :-)
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">> Florian: Welcome to the Marble Team! - Great to have you on board - I hope you don't feel discouraged by my comments :-)
Thanks!
No, I don't feel discouraged by your comments :-) As a newbie in open source developement, I'm glad about the feedback. These patches mostly were some quick hacks I primary put together to fit my own needs but then decided to share, so others can profit from it too.
As soon as I have some spare time again (probably later this week), I'll go and fix the open issues.</pre>
<br />
<p>- Florian</p>
<br />
<p>On September 6th, 2011, 1:50 p.m., Florian Eßer 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 Florian Eßer.</div>
<p style="color: grey;"><i>Updated Sept. 6, 2011, 1:50 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;">The patch from http://mail.kde.org/pipermail/marble-bugs/2011-September/001532.html
==============
Hi,
I had the same problem myself recently.
So here's a patch that adds an option "Adjust track color" in the
track's context menu just below "Export route".
Todo / Possible enhancements:
* make all the different route types (e.g. deviated from route)
configurable, not just the standard one. Right now, only the alpha
values get adjusted for all track types/colors.
* save/load to config file
** and then: button to restore the nice marble defaults if you have
experimented to much ;-)
Greetings
Florian
On 28.08.2011 09:48, Robin Seidel wrote:
> hi, thanks for the great marble!!! I have one wish: could you allow
> to adjust the transparency of the route overlay, so one can see the
> track beneath, this would be especially helpful for walking routes,
> where one may not want to take the smallest paths. Once the route is
> found one can currently not easily see what's beneath.
>
> best regards
>
> robin
</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;">worksforme</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/routing/RoutingLayer.h <span style="color: grey">(8610f18)</span></li>
<li>src/lib/routing/RoutingLayer.cpp <span style="color: grey">(3c8adfa)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102540/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>