<table><tr><td style="">rkflx created this revision.<br />rkflx added reviewers: dhaumann, ngraham.<br />Restricted Application added a project: Frameworks.<br />Restricted Application added a subscriber: kde-frameworks-devel.<br />rkflx 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/D14501">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">General</span></span></span> tab of KIO's properties dialog shows labels on the<br />
left side, and properties on the right, creating a table-like structure.<br />
Commonly a label should be aligned vertically in such a way it matches<br />
the first line of the properties, to make it clear which properties it<br />
belongs to.</p>

<p>However, this was not yet the case for the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Size</span></span></span> label.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>In Dolphin, select a folder and press <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">Alt</kbd><span class="kbd-join" style="padding: 0 4px; color: #92969D;">+</span><kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">Return</kbd>. All labels in<br />
<span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Properties</span></span><span style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">General</span></span></span> should be top-aligned.<br />
Before: <a href="https://phabricator.kde.org/F6165126" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6165126: kio-properties-before.png</a><br />
After: <a href="https://phabricator.kde.org/F6165125" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6165125: kio-properties-after.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>kpropertiesdialog-alignment-fix (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14501">https://phabricator.kde.org/D14501</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/widgets/kpropertiesdialog.cpp</div></div></div><br /><div><strong>To: </strong>rkflx, dhaumann, ngraham<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>