<table><tr><td style="">sander updated this revision to Diff 38128.<br />sander added a comment.<br />Restricted Application added a subscriber: okular-devel.
</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/D7962">View Revision</a></tr></table><br /><div><div><p>Implemented all suggestions.  In particular, the new controls disappear now unless the experimental QPrinter option or 'force rasterization' is set.  The code for this is not particularly elegant, and I'd be interested to hear from the Qt pros about how it should really be done.</p>

<p>I did not change the actual option names, because I still don't see a concensus on these names anywhere.  I still like my names, but I don't object to renaming in principle.</p></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/D7962?vs=19846&id=38128">https://phabricator.kde.org/D7962?vs=19846&id=38128</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7962">https://phabricator.kde.org/D7962</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>generators/poppler/generator_pdf.cpp</div></div></div><br /><div><strong>To: </strong>sander, Okular, aacid<br /><strong>Cc: </strong>okular-devel, cfeck, rkflx, michaelweghorn, ngraham, aacid<br /></div>