<table><tr><td style="">ngraham accepted this revision.<br />ngraham added reviewers: Frameworks, cfeck.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D15000">View Revision</a></tr></table><br /><div><div><p>Works great and looks fine to me, and the code seems sensible too given that this dialog is currently implemented with a GridLayout. I'd like to see this ported to use a FormLayout at some point, which is both more semantically appropriate, and would also simplify this type of code. But that's material for another patch of course. :)</p>

<p>Please wait for at least one more review before landing.</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/D15000">https://phabricator.kde.org/D15000</a></div></div><br /><div><strong>To: </strong>shubham, dfaure, broulik, ngraham, Frameworks, cfeck<br /><strong>Cc: </strong>ngraham, broulik, kde-frameworks-devel, michaelh, bruns<br /></div>