<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 October 27th, 2011, 6:16 p.m., <b>Bernhard Beschow</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;">Thanks for the patch, having the route colors adjustable rather than hardcoded is a very good idea. Having applied your patch and seeing the rather complicated color selection dialog, however, I wondered whether it was sufficient to just set a (single) alpha transparency value, as requested in the bug report. As an additional benefit, this would also prevent regular users to "mess" with the routing colors, which have been choosen to be distinct from the tracking etc. ones (such that they make up a "color theme").
What I suggest is therefore the following: Keep the colors "soft"-coded from the config file, such that computer-savvy users can still change the colors. For regular users, there should only be a single alpha transparency selector in the config dialog, setting a common transparency for all three routing colors.
What do you think?</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;">What is complex about the color selection? We do it the same way for the other plugins from what I can see ...
</pre>
<br />
<p>- Torsten</p>
<br />
<p>On October 26th, 2011, 9:41 a.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 Oct. 26, 2011, 9:41 a.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;">I haven't tested it it with the Qt-only version yet. (--> kcfg ??)</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.cpp <span style="color: grey">(77af021)</span></li>
<li>src/lib/routing/RoutingManager.h <span style="color: grey">(cbe70bf)</span></li>
<li>src/lib/routing/RoutingManager.cpp <span style="color: grey">(7cdffa1)</span></li>
<li>src/lib/routing/RoutingProfilesWidget.h <span style="color: grey">(e2d7333)</span></li>
<li>src/lib/routing/RoutingProfilesWidget.cpp <span style="color: grey">(88c880e)</span></li>
<li>src/lib/routing/RoutingSettingsWidget.ui <span style="color: grey">(27849c2)</span></li>
<li>src/marble.kcfg <span style="color: grey">(06277c8)</span></li>
<li>src/marble_part.cpp <span style="color: grey">(3f26b50)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102540/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/102540/s/318/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/10/26/screenshot_400x100.png" style="border: 1px black solid;" alt="Routing settings with new color buttons." /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>