<table><tr><td style="">sander 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/D7949" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Hi, thanks again for the review.  I just updated the patch to include most of your suggestions.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>>  I tagged this option as 'experimental' in the GUI.<br />
 Having it only in the tooltip is definitely not enough, this should be directly in the combobox. The tooltip could contain additional information (Which > features are missing? What rendering problems are expected? Are bug reports / contributions encouraged?). Optionally add a longer "What's this?" help.</p></blockquote>

<p>I implemented  a longer "What's this" help, but I couldn't make it appear in the GUI.  What's the magic button for that?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Therefore, I would highly recommend to only enable Arthur if Poppler is recent enough (add an #ifdef).</p></blockquote>

<p>Poppler 0.60 or a Windows OS is required now.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I took <a href="https://arxiv.org/pdf/1708.01455.pdf" class="remarkup-link" target="_blank" rel="noreferrer">https://arxiv.org/pdf/1708.01455.pdf</a> and printed to a "PDF file" (can be observed in an Arthur based Okular, too):</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">page 21: parts of vector (?) illustration missing</pre></div></blockquote>

<p>A patch for this can be found at <a href="https://bugs.freedesktop.org/show_bug.cgi?id=103118" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.freedesktop.org/show_bug.cgi?id=103118</a></p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">large filesize (5x), because of letters being vectors instead of fonts</pre></div></blockquote>

<p>Yeah, I know about this one.  There's nothing really do be done about this on the Okular side, this is Qt's doing.<br />
On the other hand one could argue that this is what "printing to a file" actually means: you lose all semantic<br />
information like letters, annotations, tags, etc.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>(might lead to problems regarding the printer's cpu/memory resources?)</p></blockquote>

<p>No idea.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7949" rel="noreferrer">https://phabricator.kde.org/D7949</a></div></div><br /><div><strong>To: </strong>sander, Okular<br /><strong>Cc: </strong>ltoscano, rkflx, michaelweghorn, ngraham, aacid<br /></div>