<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 />
<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>
<br />
<p>- Torsten</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>