<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D20735">View Revision</a></tr></table><br /><div><div><p>Code looks sane now. But the UI needs polish. Putting it in the advanced  page feels awkward to me, and even on that page, its location isn't right. It feels arbitrary, disconnected from the rest of the UI. I feel like it might sit better on the main page, to the left of the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Advanced Permissions</span></span></span> button. Alternatively, if it has to stay on the advanced page, it needs to feel better integrated with the rest of the layout.</p>

<p>Adding <a href="https://phabricator.kde.org/tag/vdg/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">#VDG</a> for more commentary.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20735">https://phabricator.kde.org/D20735</a></div></div><br /><div><strong>To: </strong>shubham, ngraham, pino, VDG<br /><strong>Cc: </strong>mmustac, pino, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>