<table><tr><td style="">ngraham added a reviewer: Frameworks.<br />ngraham accepted this revision.<br />ngraham added a comment.
</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/D12311">View Revision</a></tr></table><br /><div><div><p>Much better! I'd still prefer less whitespace, but this is already an improvement. But now that I'm staring at this dialog over and over again, I'm wondering if we even need the non-bold caption text at all. It just repeats the bold text in a slightly more awkward and more wordy fashion. What do you think?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R121 Policykit (Polkit) KDE Agent</div></div></div><br /><div><strong>BRANCH</strong><div><div>align-lock-icon (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12311">https://phabricator.kde.org/D12311</a></div></div><br /><div><strong>To: </strong>sharvey, davidedmundson, ngraham, abetts, Frameworks<br /><strong>Cc: </strong>davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>