<table><tr><td style="">rkflx planned changes to this revision.<br />rkflx 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/D6696" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Thanks Nate, you make a rational point (even if I do not agree fully). As promised above:</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 will now try to find other reviewers. If they find the single edge case pictured above to be more important than the general case (e.g. non-thumbnail sidebar, different document aspect ratios, users acting rationally) we could try to limit the sidebar width to half the window size.</p></blockquote>

<p>…which I will now add to my queue of things to try to implement (won't be trivial, as the window size can change at runtime).</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/D6696" rel="noreferrer">https://phabricator.kde.org/D6696</a></div></div><br /><div><strong>To: </strong>rkflx, Okular, aacid, sander, pino<br /><strong>Cc: </strong>ngraham, pino, sander, aacid<br /></div>