<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 14, 2015 at 1:43 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">On 14 July 2015 at 14:38, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
><br>
><br>
> On Tue, Jul 14, 2015 at 1:29 PM, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On 14 July 2015 at 13:19, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
>> ><br>
>> > On Jul 14, 2015 12:15 PM, "Lubomir I. Ivanov" <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>><br>
>> > wrote:<br>
<span class="">>> >><br>
>> >> On 14 July 2015 at 00:16, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
>> >> ><br>
>> >> > Should I convert the dive profile to QImage during previewing only or<br>
>> >> > should<br>
>> >> > I convert it during actual printing also which will affect the<br>
>> >> > printing<br>
>> >> > quality?<br>
>> >> ><br>
>> >><br>
>> >> i can't build ATM, but i think the logic here is a bit wrong:<br>
>> >><br>
>> >><br>
>> >> <a href="https://github.com/Gehadelrobey/subsurface/commit/d5b9e8424f82f4960d44a2f16abda1cbf27d7673" rel="noreferrer" target="_blank">https://github.com/Gehadelrobey/subsurface/commit/d5b9e8424f82f4960d44a2f16abda1cbf27d7673</a><br>
>> >><br>
</span><span class="">>> >> "We must pass a QPaintDevice with type QPixmap for previewing and with<br>
>> >> type QPrinter for actual printing."<br>
>> >><br>
>> >> does that include QPrintPreviewDialog as well? if so that's wrong.<br>
>> >> you can use a QPixmap if you are rendering an image to be shown in the<br>
>> >> template edit dialog, but the actual QPrintPreviewDialog contents<br>
>> >> should be pretty much the same as the printed contents (on a hardcopy<br>
>> >> or in a PDF).<br>
>> >><br>
>> ><br>
>> > No, the preview function is used for the QPixmap in the TemplateEdit<br>
>> > only,<br>
>> > while the QPrintPreviewDialog uses the actual print() function.<br>
>><br>
>> both the preview and print profiles are in vector for me, which is good.<br>
>> also the color and edit seems to be working and the preferred colors<br>
>> are stored (in the settings/registry(win32), apparently).<br>
>><br>
>> we *might* have to get some user feedback on the colors...i think<br>
>> storing them globally is a bad idea.<br>
>> will create a thread on the ML for that.<br>
>><br>
>> 1) the "per-template" vs "global colors" issue..<br>
><br>
><br>
> Sorry, what do you mean by global colors? do you mean the almond_colors<br>
> instance of the color struct?<br>
><br>
<br>
</span>ignore this for now. just give the colors proper names in the dialog<br>
(e.g. "Background" etc..)<br>
<span class=""><br></span></blockquote><div><br>Hello Lubomir,<br><br>I have fixed the names in the TemplateEdit dialog as you have suggested, I have also fixed the QPrintPreviewDialog grayscale issue and pushed them to my branch.</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>