<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 1, 2015 at 10:20 AM, 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On 1 July 2015 at 03:24, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
><br>
><br>
> On Tue, Jun 30, 2015 at 11:13 AM, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On 30 June 2015 at 03:53, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
>> ><br>
>> ><br>
>> > On Mon, Jun 29, 2015 at 4:13 PM, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> On 29 June 2015 at 05:08, Gehad Elrobey <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
>> >> > Hello all,<br>
>> >> ><br>
>> >> > This week I was working on the list of tasks which I have discussed<br>
>> >> > with<br>
>> >> > Lubomir.<br>
>> >> ><br>
>> >> > Tasks completed so far:<br>
>> >> > - QPrintPreviewDialog triggered from the main dialog<br>
>> >><br>
>> >> nice<br>
>> >><br>
>> >> > - add "print selected dives" support<br>
>> >><br>
>> >> this one doesn't work for me. i have a massive test list of dives and<br>
>> >> with "print selected dives" not selected, i expect it to print the<br>
>> >> entire list, instead it prints only a single page with two dives.<br>
>> >><br>
>> ><br>
>> > That was working for me before I merged commit 094264014dc47, it seems<br>
>> > that<br>
>> > amount_selected in display.h is not working as expected, I will look<br>
>> > into<br>
>> > this and push a commit.<br>
>> ><br>
>><br>
>> ok<br>
>><br>
>> >><br>
>> >> > - print in colors or gray scale<br>
>> >><br>
>> >> does not work. the profile is still printed in color no matter the<br>
>> >> print dialog selection.<br>
>> >> do you simply attempt to greyscale the profile image before rendering<br>
>> >> on the HTML?<br>
>> ><br>
>> ><br>
>> > I use QPrinter::setColorMode() which selects the color mode for the<br>
>> > printer<br>
>> > to be RGB or Gray scale, anyway the output in the QPrintPreviewDialog is<br>
>> > always in RGB mode, while actual printing using the printer is working<br>
>> > correctly for me.<br>
>> ><br>
>><br>
>> greyscale should work for both the preview and the actual print.<br>
>><br>
><br>
> I think QPrintPreviewDialog reflects page related options only, so some<br>
> QPrinter related settings like colorMode and resolution can't be previewed<br>
> with the dialog.<br>
><br>
<br>
</div></div>how so?<br>
in *all* of the previous implementations both resolution and colorMode<br>
were visible/reflected on the preview!<br>
<br></blockquote><div><br>I tested the color mode selection in the old printing code and it was not working in QPrintPreviewDialog, so this may be a QT5 bug?<br>I tested with both v4.4.2 and commit f8a3a85 and it didn't work in any of them.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
both PrintDialog::onPaintRequested() and PrintDialog::printClicked()<br>
call the same method Printer::print() which then calls<br>
Printer::render() and greyscale mode becomes optional.<br>
<br>
what am i missing?<br>
<span class=""><br></span></blockquote><div><br>I am doing exactly the same thing, and I call QPrinter::setColorMode(GrayScale) and I call Profile::<span style="color:rgb(0,0,0)">setPrintMode(), am I missing something?</span><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">
> Also as you have mentioned that profile2 is missing grey scale capability so<br>
> I think we may need to remove 'print in color' check box which is in my<br>
> opinion useless, because user can still select printing color mode from<br>
> option tab in QPrintDialog which will affect the actual printing and not the<br>
> previewDialog.<br>
><br>
><br>
<br>
</span>i disagree...the Preview should be exactly the same as the Print output.<br>
the profile2 custom color table is also doable but for now we should<br>
simply use QPrinter::GrayScale.<br>
<br></blockquote><div><br>I thing a solution for this we can use css to turn the whole page to gray scale before rendering also we will need to use the custom color table, what do you think about this?<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
BTW, Gehad we had a short discussions with the other GSoC mentors and<br>
we came to the conclusion that some of the students are not very<br>
responsive.<br>
i know that i'm not on IRC, and i'm not saying that you don't respond<br>
at all, but could you try to be *more* responsive via email i.e. write<br>
immediately after you see an email, cover all questions etc...this is<br>
important for your final GSoC score.<br></blockquote><div><br></div><div>Sorry for that I will be more responsive to further emails.</div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">regards,</div></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div><div dir="ltr"><div>Gehad</div></div></div></blockquote></div></div>
</div></div>