<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><p>Thanks for looking at my patch, but I'm not sure I'm understanding you completely here (you didn't state a reason for your preference). How would this fix the header text being cut off and the (seemingly) arbitrary max width restriction? Hardcoding any fixed widths (even with HiDPI scaling in mind) seems wrong.</p>

<p>Removing the lines, the sidebar adapts its width to the header text. In practice, apart from eliminating visual glitches, only little will change, e.g. for English, both default and minimum sidebar widths are nearly identical (if not a little smaller).</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>aacid<br /></div>