<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added reviewers: Plasma, Frameworks.<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/D15644">View Revision</a></tr></table><br /><div><div><p>Thank you for the patch! I see that this is your first KDE contribution, how exciting! This needs some work, so please don't get discouraged, and I'll help you where I can. Here are the problems that need to be solved in order for the patch to be landable:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Please see <a href="https://community.kde.org/Infrastructure/Phabricator#Formatting_your_patch" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Infrastructure/Phabricator#Formatting_your_patch</a> for information regarding how to format your patch's title and summary sections.</li>
<li class="remarkup-list-item">The patch doesn't apply and I don't see how it could work; all is does is remove a comment. The patch you attached to <a href="https://bugs.kde.org/show_bug.cgi?id=395349" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=395349</a> is substantially different; can you edit this revision to have that diff instead? To do that, you can click on the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Edit Revision</span></span></span> link in the top-right corner of the page (it may be under the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Action</span></span></span> button if your window is narrow.</li>
</ol></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15644">https://phabricator.kde.org/D15644</a></div></div><br /><div><strong>To: </strong>lsartorelli, ngraham, Plasma, Frameworks<br /><strong>Cc: </strong>ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>