<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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 gave this a try today and it works nicely! Awesome work! I'd still like some user interface improvements though, because gating these scaling options behind <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 not very user friendly: the connection between that setting and the scaling options is not likely to be apparent to any user who does not also happen to be a PDF expert or an Okular developer. :)</p>

<p>Here are some ideas to make the presentation a bit more user-friendly:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Since the image always needs to be rasterized before the new scaling options work... why don't we keep the scaling options enabled and turn on rasterization automatically when they're used? Basically we just remove the manual step of making the user check <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> if they want to scale the document. We assume that if they want to scale the document, they're willing to accept whatever technical changes are required.</li>
<li class="remarkup-list-item">When using <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Scale to: Full page</span></span></span>, it might be nice if some warning text could appear below the combobox notifying the user that the document may get cut off at the edges if it does not include its own margins.</li>
<li class="remarkup-list-item">What is the relationship between the two options? For example, if I choose <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Scale Mode: None</span></span></span> and <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Scale to: Printable Area</span></span></span>, I don't have a clear picture of what will happen. The first one seems to imply that there will be no scaling, but the second one allows me to choose what the scaling target will be. It seems like  <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Scale Mode: None</span></span></span> should disable the second one, but it doesn't.</li>
</ul></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>okular-devel, cfeck, rkflx, michaelweghorn, ngraham, aacid<br /></div>