ngraham added a comment.
>> 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 Force rasterization 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. > > To be honest, this sounds like even more magic than what is happening right now. And it will become even more confusing if we ever merge https://phabricator.kde.org/D7949 . Alternative suggestion: I'll let the tooltip hint at the rasterization option. Is there a good UI reason to force users to manually click "force rasterization", or is it just technical difficulty? > > >> When using Scale to: Full page, 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. > > Isn't that obvious? Besides, there are other ways to cut off your document too (e.g. print A3 document onto A4 paper with scaling turned off). Hm, it wasn't obvious to me. >> What is the relationship between the two options? For example, if I choose Scale Mode: None and Scale to: Printable Area, 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 Scale Mode: None should disable the second one, but it doesn't. > > The difference between 'none, printable area' and 'none, full page' is the position on the page where the document is printed too. I'm afraid that still doesn't help me understand what will happen. :) I would expect that Scale Mode: None would result in no scaling at all, which is why it's confusing to then have another control to change the type of scaling that I'm under the impression won't be applied. REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D7962 To: sander, #okular, aacid, #vdg, ngraham Cc: okular-devel, cfeck, rkflx, michaelweghorn, ngraham, aacid
