<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 28, 2015 at 7:35 PM, Lubomir I. Ivanov <span dir="ltr"><<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 28 July 2015 at 19:26, Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
> On Tue, Jul 28, 2015 at 07:09:37PM +0300, Lubomir I. Ivanov wrote:<br>
>> > In my case the "toggle calculated ceilings" setting is on (as I want to<br>
>> > see those) and strangely on the screen the "GF 30/70" doesn't bother me at<br>
>> > all, but on the printout I find it odd on dives that don't have a visible<br>
>> > ceiling.<br>
>> ><br>
>> > So what someone needs to do is change the profile code to do the right<br>
>> > thing and only show that text if there actually is a ceiling shown.<br>
>> ><br>
>><br>
>> ok, i understand.<br>
>> (for Gehad, to enable the ceiling graph click on the topmost toolbar<br>
>> button next to the profile)<br>
>><br>
>> "printMode" in the profile widget doesn't hide/handle the ceiling<br>
>> graph and text, so i think the issue is different here.<br>
><br>
> Yeah, I think we just should either not print the GF at all in print mode<br>
> or at least only print it if there is any ceiling.<br>
><br>
>> for instance, test14.xml with the "Two dives" printing template has<br>
>> the calc. ceiling graph so small that's it's not visible, while only<br>
>> the GF X/Y text is visible.<br>
>><br>
>> are you experiencing the same issue?<br>
><br>
> That's part of it. The bigger issue for me is that GF x/y is printed even<br>
> if there is no ceiling at all. And that's kinda strange.<br>
><br>
<br>
</span>i have no idea how are we going to hide the GF text based on no how<br>
small the ceiling graph is.<br>
but this is generic profile problem, as i'm able to visually reproduce<br>
that in the application window as well by resizing the profile to be<br>
way too small.<br>
(i.e. GF visible, ceiling not visible).<br>
<span class=""><br>
> BTW: I have added SampleDivesV2.ssrf a while ago that contains a more<br>
> representative selection of dives. I think that one is better for testing<br>
> of the user experience than the test dives, many if which were added ages<br>
> ago to test a specific new feature and are kind of odd in other ways...<br>
><br>
>> >> > - having the warning triangles but no way to see their text... I think<br>
>> >> >   we should just not print the triangles. Opinions?<br>
>> >><br>
>> >> same as the previous printer code. perhaps we should simply hide them<br>
>> >> with "printMode" (as Miika suggests) for now?<br>
>> ><br>
>> > Yes. Please.<br>
>><br>
>> how about the other icon with mouse over tooltip text like ":flag"?<br>
><br>
> As Miika said, the gas change icons make sense. The flag for bookmark<br>
> maybe makes sense, too. The triangles are pointless without explanation.<br>
><br>
<br>
</span>ok, only hiding the triangles for now.<br>
Gehad, basically Dirk wants us hide the warning! triangles on the<br>
profile when in "printMode".<br>
to do that, DiveEventItem::setupPixmap() has and example of which icon<br>
is used based on the event name/type.<br>
<br>
to skip the rendering of an event icon go to<br>
ProfileWidget2::plotDive() and find the following loop:<br>
while (event) {<br>
   DiveEventItem *item = new DiveEventItem();<br>
   ....<br>
<br>
this item needs not to be created if it's setPixmap() method is about<br>
to use ":warning" and if printMode=true.<br>
<br>
SampleDivesV2.ssrf/test14.xml have warning triangles in the profile.<br>
also please, go through the thread and grab the rest of the feature requests.<br>
<br></blockquote><div><br>Okay Lubomir, I'll add them to my todo list.<br></div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">regards,</div></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div dir="ltr"><div>Gehad</div></div></div></blockquote></div></div>
</div></div>