<div dir="ltr">Hey folks,<div><br></div><div>the printsupport in Qt has still couple of open tasks, forwarding a mail from John. The Qt 5.2 feature freeze is very close now, I'll be working on those this week, but if anyone wants to help, feel free to claim some tasks. I'll be adding them to the wiki shortly, so be sure to claim it there, so others know you're doing them.</div>

<div><br></div><div>Also, David & Kevin, see John's last point, what's your take on that?</div><div><br></div><div>Cheers</div><div>-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>

<br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">John Layt</b> <span dir="ltr"><<a href="mailto:jlayt@kde.org">jlayt@kde.org</a>></span><br>Date: Sun, Sep 8, 2013 at 9:07 PM<br>

Subject: Print Dialog - next steps<br>To: Martin Klapetek <<a href="mailto:martin.klapetek@gmail.com">martin.klapetek@gmail.com</a>>, Rohan Garg <<a href="mailto:rohan@kde.org">rohan@kde.org</a>><br><br><br>Hi guys,<br>


<br>
Your code is looking good, just that design change to take into account and we<br>
should be done with those two reviews.<br>
<br>
There's three more tasks after that, most of which are adapting more of my old<br>
KDE code:<br>
<br>
1) Add Banner Page support to the Job tabs.  Take the Banner Pages box from<br>
the KDE Pages tab and add it to Martin's Job tab in Qt.  Should be quick and<br>
easy to do now.<br>
<br>
2) Add Page Range support for apps that can't do it themselves.  Some apps<br>
like Kate/KWrite can't select pages to print, so just print the entire<br>
document.  Cups can select the pages for them.  In KDE we provide api for the<br>
apps to set to use this, but we can do it implicitly in Qt without api.  If<br>
the app doesn't set PrintDialogOption::PrintPageRange then we don't normally<br>
show that Page Range entry.  If we have Cups then we can show it, tell the app<br>
to print AllPages, and tell Cups the page range to print.<br>
<br>
3) Add Page Set support.  We currently insert the Page Set combo into the Qt<br>
dialog on the Copies tab using a dirty hack, now we can put it there for real<br>
:-)  We'll use only the Server-Side Page Selection for this, i.e. tell the app<br>
to print all the pages in the chosen range, then tell Cups to print only the<br>
odd or even pages from that.  One problem my current code deals with is that<br>
Cups applies the Page Set it to the pages it receives, not the page numbers in<br>
the app, so there's an adjustment that needs to be made first.  For example, if<br>
you print from page 4 to 8 and tell Cups to print Even Pages you will actually<br>
get pages 5 and 7.<br>
<br>
4) Cups is a bit silly in applying the PagesPerSheet setting to the physical<br>
output pages and not the logical pages.  This means if you use it with PageSet<br>
or the Cups PageRange selection the user doesn't get what they expect.  We<br>
need to make these features mutually exclusive.  Best way I think is if the<br>
user hits the Print button then we check if both have been selected and<br>
display a message box if they have.<br>
<br>
5) In KF5 the KdePrintDialog stuff can either be removed entirely, or more<br>
conservatively modified to no longer insert the extra KDE Cups widgets and<br>
modifications so they don't appear twice.  Discuss with Kevin and David which<br>
they would prefer.<br>
<br>
Hopefully now you guys are used to the code and Qt standards we can fit those<br>
in before the freeze.<br>
<br>
Cheers!<br>
<span class=""><font color="#888888"><br>
John.</font></span></div>
</div></div>