<table><tr><td style="">marten created this revision.<br />marten added a reviewer: Plasma.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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/D8424" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The current appearance of this dialogue:</p>

<p><a href="https://phabricator.kde.org/F5443493" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5443493: folderview-properties-location_BEFORE.png</a></p>

<p>has a number of visual anomalies:  most significantly, the spacing and alignment between the 3rd/4th radio buttons and their text is not the same as the 1st/2nd buttons.  Not only is the visual appearance different but also the operation:  the 1st/2nd buttons can be selected by clicking on the label in addition to the button, so increasing the target area, but the 3rd/4th can only be clicked on the actual button.  Also, the group box titles ("Location" and "Title"), with no visual distinction and indented a long way, are not obvious for being group titles.</p>

<p>The revised layout:</p>

<p><a href="https://phabricator.kde.org/F5443501" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5443501: folderview-properties-location_AFTER.png</a></p>

<p>uses a single grid layout, so aligning all of the combo boxes and entry fields.  All four radio buttons have text, so they are able to be selected by clicking on the text.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built plasma-desktop with this change.  Checked appearance and operation both as a desktop containment (when the "Title" controls do not appear) and as a standalone plasmoid (when they do).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8424" rel="noreferrer">https://phabricator.kde.org/D8424</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/ConfigLocation.qml</div></div></div><br /><div><strong>To: </strong>marten, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>