<table><tr><td style="">ngraham 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/D7962">View Revision</a></tr></table><br /><div><div><p>I'm afraid I cannot agree that this should be fixed with documentation. Users don't read documentation. And they shouldn't have to read documentation to access basic features like print output scaling. The interface needs to hide unnecessary implementation details and be as self-documenting as possible. This current patch does neither: it demands that users understand details of Qt's PDF rendering implementation ("You need to have "Force Rasterization" turned on) and offers no clues that this is connected to the hidden scaling features.</p>

<p>I'm very happy that the scaling features are being implemented, and I applaud the work you've done so far! But features that are not discoverable because they are hidden behind technical proficiency tests might as well not exist for 99% of users since they will never be discovered. I think we can do a better job on the user interface. I'm happy to work with you on this, or if you're thoroughly sick of me by now (😜), we can involve other members of <a href="https://phabricator.kde.org/tag/vdg/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">#VDG</a>. Offering such guidance is one of the big reasons why VDG exists, in fact.</p>

<p>Probably the minimum change I would want to approve this would be a label beneath the disabled scaling controls that says "Turn on 'Force Rasterization' to enable scaling options" when <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Force Rasterization</span></span></span> is turned off. But I still think we can do even better: in particular, I still don't really understand what the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Scale Mode</span></span></span> options do.</p>

<p>I know it's been a really long road for this patch, but I think we can get all the way to a nice UI without too much more effort!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></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>To: </strong>sander, Okular, aacid, VDG, ngraham<br /><strong>Cc: </strong>bruns, okular-devel, cfeck, rkflx, michaelweghorn, ngraham, aacid<br /></div>