<table><tr><td style="">michaelweghorn updated this revision to Diff 49510.<br />michaelweghorn edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-4o24mbtl7o2p2ex/">(Show Details)</a><br />michaelweghorn added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10974">View Revision</a></tr></table><br /><div><div><p>Adapt according to Albert's feedback:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">undo ABI breakage by changing back signature of 'Document::printConfigurationWidget()' to  what it is like without this change, and use a 'dynamic_cast' in 'Part::slotPrint'.</li>
<li class="remarkup-list-item">drop 'virtual' for 'PDFOptionsPage::ignorePrintMargins()'</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>class.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">In order to avoid an ABI breakage, the return value of<br />
'Document::printConfigurationWidget' is left as a 'QWdiget *'<br />
and a dynamic_cast is done on use.<br />
<br />
</span>FilePrinter is adapted to take into account the value set by<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D10974?vs=49223&id=49510">https://phabricator.kde.org/D10974?vs=49223&id=49510</a></div></div><br /><div><strong>BRANCH</strong><div><div>michaelweghorn/WAITING_FOR_REVIEW_D10974_add_scaling_options_for_other_generators</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10974">https://phabricator.kde.org/D10974</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
core/document.cpp<br />
core/document.h<br />
core/fileprinter.cpp<br />
core/printoptionswidget.cpp<br />
core/printoptionswidget.h<br />
generators/poppler/generator_pdf.cpp<br />
generators/poppler/generator_pdf.h<br />
interfaces/printinterface.h<br />
part.cpp</div></div></div><br /><div><strong>To: </strong>michaelweghorn, Okular<br /><strong>Cc: </strong>fvogt, rkflx, arthurpeters, ltoscano, okular-devel, aacid, ngraham, tfella, darcyshen<br /></div>