<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/110003/">http://git.reviewboard.kde.org/r/110003/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 14th, 2013, 10:07 p.m. UTC, <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Can you please describe your best fit algorithm?
I am finding that in my 1920x1080 resolution best fit is giving me a "Fit to width" while in evince i get a "Fit to page" that "may" actually make more sense since i am seeing more info in that "Fit to page" than in this "Fit to width" and i can still read it.</pre>
</blockquote>
<p>On April 29th, 2013, 7:16 p.m. UTC, <b>Thomas Fischer</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">My best-fit code switches between three zoom types depending on the relation of aspect ratio of Okular's area to show content and the aspect ratio of the shown page. Depending on this relation, it assumes that one of the following three cases is true:
(1) Both ratios are very similar, e.g. when you are looking on landscape-oriented PDF slides in a maximized Okular window (4:3 aspect ratio). Fit-page is the best viewing mode here.
(2) You have have a wide Okular window (e.g. maximize on a wide screen) viewing a portrait-oriented A4 document such as a letter. If you eye sight and your screen is good, you can still read it, but don't assume everyone else can. So a better choice may be to fit-width zoom the page, even if only a small section is visible, but at least the text should be readable.
(3) You have a split-screen Okular window (e.g. right or left half of your screen), but you are looking at a landscape-oriented document. Again, it may be a better choice to fit-height zoom your document than to fit-page zoom it if you want to read the fineprint.
On extreme cases, best-fit may zoom in a way that the document becomes even hard to read, but in most cases it has its benefits being a good approximation for typical use-cases (in my personal view).</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Do you think you could take a few screenshots of the various scenarios with various kind of document formats in both okular+your patch and evince and post them to forums.kde.org so that people can comment (even the forums have poll support IIRC) which of the two "Best Fit" algorithms they like more? If we're going to do a Best Fit, lets make sure it's "Best Fit" by all people's opinion and not just based or your or my personal view.</pre>
<br />
<p>- Albert</p>
<br />
<p>On April 14th, 2013, 11:26 a.m. UTC, Thomas Fischer wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Okular.</div>
<div>By Thomas Fischer.</div>
<p style="color: grey;"><i>Updated April 14, 2013, 11:26 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Attached patch implements best-fit zoom for Okular. It is a refined version of the patch submitted in bug report 249364, attachment 51069. The refinement addresses the scrollbar issues as observed in continuous view mode.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=249364">249364</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>part.rc <span style="color: grey">(64aeffb)</span></li>
<li>ui/pageview.h <span style="color: grey">(5e839f2)</span></li>
<li>ui/pageview.cpp <span style="color: grey">(6e093ef)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110003/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>