D10974: Add option to ignore print margins

2018-07-10 Thread Michael Weghorn
michaelweghorn added a comment. @arthurpeters: Thanks for your ideas on this. For now, I'd like to stick with the two options that have been discussed so far in this change (see reasoning in D10974#217969 ) and suggest to handle potential further i

D7949: Allow to print pdf doc directly into a QPrinter

2018-07-10 Thread Michael Weghorn
michaelweghorn added a comment. In general, I really like the idea of providing this new option. Two notes after having a quick look (besides the two comments directly on the code): - Depending on what the use case for the option is (Will users who want to use this experimental feat

D10974: Add option to ignore print margins

2018-07-10 Thread Nathaniel Graham
ngraham added a comment. In D10974#290127 , @rkflx wrote: > - `Scale to:` `Printable area` | `Paper size` +1, I would be fine with this. REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D10974 To: michaelweghorn, #

D10974: Add option to ignore print margins

2018-07-10 Thread Henrik Fehlauer
rkflx added a comment. FWIW, earlier in D7962 there was this suggestion: In D7962#150260 , @rkflx wrote: > > Scale mode: fit-to-page vs. shrink-to-page vs. none > > Scale to: printable area vs. full page

D7949: Allow to print pdf doc directly into a QPrinter

2018-07-10 Thread Henrik Fehlauer
rkflx accepted this revision. rkflx added a comment. This revision is now accepted and ready to land. Thanks, accepting the Diff from my side (obviously I cannot speak for the Okular project). In D7949#290074 , @sander wrote: > That is https:

D7949: Allow to print pdf doc directly into a QPrinter

2018-07-10 Thread Oliver Sander
sander marked 3 inline comments as done. sander added a comment. >>Unlike for CUPS printing, Qt printing prints on the entire page and does not scale to the printable area yet. This is only because it is the easiest way. I > >plan to implement a few standard scaling methods in a subsequent p

D7949: Allow to print pdf doc directly into a QPrinter

2018-07-10 Thread Oliver Sander
sander updated this revision to Diff 37522. REPOSITORY R223 Okular CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7949?vs=37521&id=37522 REVISION DETAIL https://phabricator.kde.org/D7949 AFFECTED FILES generators/poppler/generator_pdf.cpp To: sander, #okular Cc: okular-devel, a

D7949: Allow to print pdf doc directly into a QPrinter

2018-07-10 Thread Oliver Sander
sander updated this revision to Diff 37521. sander added a comment. Restricted Application added a subscriber: okular-devel. Applied Henrik's suggestions REPOSITORY R223 Okular CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7949?vs=37025&id=37521 REVISION DETAIL https://phabric