<table><tr><td style="">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><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>stupidly unusable</p></blockquote>

<p>I find it really hard to translate this kind of comment into actionable feedback I can improve upon, because this neither details an actual problem, nor does it suggest an alternative size.</p>

<p>Instead, let me expand on the properties of my proposed solution, so we can determine any flaws:</p>

<p>For the minimum size:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">determined by the widgets inside the sidebar container</li>
<li class="remarkup-list-item">large enough to still allow for usage of all functionality</li>
<li class="remarkup-list-item">small enough to not get in the way when space is tight</li>
<li class="remarkup-list-item">elision of header text if needed</li>
<li class="remarkup-list-item">in practice ~90px</li>
</ul>

<p>For the maximum size:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">not restricted to a hardcoded size</li>
<li class="remarkup-list-item">nevertheless, the actual document itself is never hidden completely</li>
<li class="remarkup-list-item">accommodates long TOC entries, reviews and bookmarks, the user gets to choose the distribution of available space to sidebar and document for herself</li>
<li class="remarkup-list-item">for horizontally wide documents, e.g. construction plans, moving the dark rectangle in the thumbnail preview with the mouse to pan over the document is easier for larger widths</li>
<li class="remarkup-list-item">if a user does not like the thumbnail being large, she can always reduce the size of the sidebar</li>
<li class="remarkup-list-item">removal of restriction is precondition if we were to implement a zoomable thumbnail grid (which seems standard in other viewers)</li>
<li class="remarkup-list-item">today's ultrawide 21:9 displays have widths of up to 3.440 pixels, Okular's sidebar should be flexible within these bounds and not restricted to 600px</li>
<li class="remarkup-list-item">no reasonable argument for restricting in commit history, HIG, bugzilla and this review (besides a single "opinion")</li>
<li class="remarkup-list-item">users are asking for it (see bug) and may have workflows and documents we currently do not cater for</li>
<li class="remarkup-list-item">other free and proprietary document viewers do not restrict the maximum sidebar size either (I tested GNOME's document viewer, Adobe Reader and Foxit Reader)</li>
<li class="remarkup-list-item">other KDE apps are unrestricted too, e.g. Dolphin, Gwenview and KMail</li>
</ul>

<p>If there are any other comments regarding my patch, please share.</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<br /><strong>Cc: </strong>sander, aacid<br /></div>