<table><tr><td style="">jbbgameich created this revision.<br />Herald added a project: Kirigami.<br />Herald added a subscriber: plasma-devel.<br />jbbgameich requested review of this revision.
</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/D25175">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>previously the height of the label was based on the content's height. However this doesn't work in non wide mode. With this change it preserves the previous behaviour in wide mode were it makes sense but uses a constant height in non-wide mode.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>FormLayouts with large content items don't have large margins between the items anymore when wideMode = false.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>fix/formlayout-label-height (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25175">https://phabricator.kde.org/D25175</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/controls/FormLayout.qml</div></div></div><br /><div><strong>To: </strong>jbbgameich<br /><strong>Cc: </strong>plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, ahiemstra, davidedmundson, mart, hein<br /></div>