<table><tr><td style="">ngraham retitled this revision from "[SimpleKCM] Base the maximum height on the screen height, not some random value" to "[RFC] [SimpleKCM] Fix height calculation".<br />ngraham edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-jkrf62vikahvk3z/">(Show Details)</a><br />ngraham edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-y6bauh5zsh73dbw/">(Show Details)</a>
</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/D20283">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">SimpleKCM currently <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">imposes a maximum</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">calculates the</span> height of <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">`Kirigami.Units.gridUnit * 20`</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">its page incorrectly in a few ways:<br />
- `flickable.contentHeight`, which determines the base height</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">which is</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">always gets set to 0</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">arbitrary</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Paddings</span> and <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">results in many KCMs opened standalone in kcmshell still getting a scrollbar</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">headers aren't taken into account</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">even when there's plenty</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- There's an arbitrary maximum height</span> of <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">vertical space available.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">`Kirigami.Units.gridUnit * 20`</span><br />
<br />
This patch <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">takes paddings and headers into account, and </span>changes the maximum <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">value </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">height<br />
</span>so that it's based on a fraction of the available<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> screen height, ensuring that tall KCMs</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">screen height, ensuring that tall KCMs </span>will fill the available space as often as possible<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">.<br />
<br />
RFC because `flickable.contentHeight` always evaluates to 0 and I can't figure out why</span>.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">CC</span>BUG: 398820<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
FIXED-IN: 5.58<br />
<br />
</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">No actual changes can be observed because `flickable.contentHeight` always evaluates to<br />
0 due to https://bugs.kde.org/show_bug.cgi?id=398820 and I haven't yet figured out the<br />
fix for that issue. But once that's fixed, SimpleKCM KCMs will have more sensible<br />
default heights when opened in kcmshell.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R296 KDeclarative</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20283">https://phabricator.kde.org/D20283</a></div></div><br /><div><strong>To: </strong>ngraham, Plasma, mart, broulik<br /><strong>Cc: </strong>broulik, davidedmundson, GB_2, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>