<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 28, 2015 at 6:09 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 18:41, Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
> On Tue, Jul 28, 2015 at 06:34:36PM +0300, Lubomir I. Ivanov wrote:<br>
>> > I am blown away. This is really great work. It's not perfect, there are<br>
>> > things that I'd like to see improved, but it is such an amazing<br>
>> > step forward for our printing infrastructure.<br>
>><br>
>> thanks, all credit goes to the student.<br>
><br>
> Of course, Gehad did most of the work, but you have done a fantastic job<br>
> mentoring him, pushing for solid solutions and helping him along. I'm very<br>
> happy with BOTH of you.<br>
><br>
>> > Things I noticed in a few minutes of testing:<br>
>> ><br>
>> > - the profiles always contain the GF on the top... that should only be<br>
>> >   there if we show any ceiling<br>
>><br>
>> my terminology is a bit lacking here. could you explain which<br>
>> preferences need to be enabled for that to be observed and which<br>
>> testXX.xml dive can be used for testing?<br>
><br>
> Great question. The printing code does something very smart - it uses the<br>
> current settings which means people see the things on paper that they see<br>
> on screen - and don't see the things that they haven't enabled.<br>
><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>
</span>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>
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></blockquote><div><br>Yes, It is mainly covered with the text.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><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>
</span>how about the other icon with mouse over tooltip text like ":flag"?<br>
<br>
lubomir<br>
--<br>
</blockquote></div><br><br clear="all"><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>