<table><tr><td style="">rkflx created this revision.<br />Restricted Application added a project: Okular.
</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><strong>REVISION SUMMARY</strong><div><p>In some languages the sidebar container header is wider than the<br />
minimum container width and therefore cut off. By removing the<br />
explicitly set minimum width, the QVBoxLayout is now allowed to<br />
actually do its job of automatically calculating the proper minimum<br />
width.</p>

<p>In times of HiDPI screens, setting the maximum width to 600px seems<br />
arbitrary and overly restrictive, so it is removed.</p>

<p>Both width restrictions were originally chosen in <a href="https://phabricator.kde.org/R223:078b81b7807d6af600474e40a8ce62525a46ce4e" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">078b81b7</a> with no<br />
explanation given for the values selected, which later were changed to<br />
different values multiple times. Today, they simply do not seem<br />
necessary anymore.</p>

<p>Note: The minimum size for some languages is now wider than before,<br />
still acceptable though (see screenshots of Greek, where the translated<br />
string of "Thumbnails" is one of the longest between all languages).<br />
Most languages are not affected at all.</p>

<p>BUG: 176780</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Use default fonts, change application language to Greek. Move splitter<br />
left (text in header of thumbnail sidebar not cut off anymore) and<br />
right (does not stop at 600px). Window resizing works as expected.</p>

<div class="remarkup-table-wrap"><table style="border-collapse: separate;
          border-spacing: 1px;
          background: #d3d3d3;
          margin: 12px 0;">
<tr><td style="background: #ffffff;
          padding: 3px 6px;">Before</td><td style="background: #ffffff;
          padding: 3px 6px;">After</td></tr>
<tr><td style="background: #ffffff;
          padding: 3px 6px;"><a href="https://phabricator.kde.org/F3810783" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F3810783: okular-min-before.png</a></td><td style="background: #ffffff;
          padding: 3px 6px;"><a href="https://phabricator.kde.org/F3810784" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F3810784: okular-min-after.png</a></td></tr>
</table></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</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>AFFECTED FILES</strong><div><div>ui/sidebar.cpp</div></div></div><br /><div><strong>To: </strong>rkflx, Okular<br /><strong>Cc: </strong>aacid<br /></div>