<table><tr><td style="">rkflx retitled this revision from "Fix cut off text in sidebar header, remove restricted max width" to "Elide cut off text in sidebar header, remove restricted max width".<br />rkflx edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-lg62a2axrqkyssw/" rel="noreferrer">(Show Details)</a><br />rkflx edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-5kyrxebshdnj3qn/" rel="noreferrer">(Show Details)</a><br />rkflx added a dependency: D7164: KSqueezedTextLabel: Respect indent, margin and frame width.
</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>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>minimum container width and therefore cut off. By <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">removing the</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">using a squeezable<br />
label and setting a proper default width, the header is displayed fully<br />
when Okular is started for the first time, while still being able to</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">reduce the size via the splitter.<br />
<br />
By removing the </span>explicitly set minimum width, the QVBoxLayout is<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> now allowed to</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">now allowed to </span>actually do its job of automatically calculating the<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> proper minimum</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">proper minimum </span>width<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> based on its contained widgets</span>.<br />
<br />
In times of <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">HiDPI screens</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">ultrawide 21:9 displays</span>, setting the maximum width to 600px<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> seems</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">seems </span>arbitrary and overly restrictive, so it is removed.<div style="padding: 8px 0;">...</div>Note: The <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">minimum</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">first-start</span> size for some languages is now wider than before,<br />
still acceptable though (see screenshots of Greek, where the translated<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
string of "Thumbnails" is one of the longest among all languages).<br />
Most languages are not affected at all and users could easily move the<br />
splitter further to the left.<br />
<br />
Note: KF5_REQUIRED_VERSION is not bumped for now to require the fix to<br />
KSqueezedTextLabel (see depending review), as this minor visual tweak<br />
should not prevent building of Okular on older distros. For those, at</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">string of "Thumbnails" is one of the longest between all languages).</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">least cutoff will be reduced to 9px, distros with updated KF5 would get</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Most languages are not affected at all</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">the full cutoff reduction to 0px</span>.<br />
<br />
BUG: 176780<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
Depends on D7164</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Use default fonts, change application language to Greek. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Move splitter</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Remove<br />
Okular's config file before restarting. Notice how sidebar header is</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">left (text in header of thumbnail sidebar not cut off anymor</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">not cut of anymore. Move splitter left (text in header should elid</span>e) and<div style="padding: 8px 0;">...</div>| Before <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">(cut off) | After (minimum elided) </span>| Afte<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">r |</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">r (first start) |</span><br />
| {F38<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">10783</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">32576} | {F3832577</span>} | {F38<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">10784} |</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">32578} |<br />
</span></div></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>