<table><tr><td style="">romangg 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/D28818">View Revision</a></tr></table><br /><div><div><p>Header subject and commit message body must have shorter line length in line with message guideline. Also note regarding capitalization <a href="https://github.com/angular/angular/blob/3cf2005a936bec2058610b0786dd0671dae3d358/CONTRIBUTING.md#subject" class="remarkup-link" target="_blank" rel="noreferrer">Angular's subject guideline</a>.</p>

<p>Also the current description sucks. That's a single sentence with way too many words and no grammar.</p>

<p>Come on, coding is not only about the code. Be proud of your work and present it to us in the way that it deserves.</p>

<p>Also my assumption is you work together with the original reviewers in the same company and may have discussed this patch internally already. That's fine but note that this project and any other KDE project is not owned by your company alone and it's your responsibility in the end to transform your internal discussions and the knowledge you gained from that into public knowledge available to all members of this community.</p>

<p>That's why a good description, and in the end all of the documentation you do when coding, is tremendously important.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R104 KScreen</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28818">https://phabricator.kde.org/D28818</a></div></div><br /><div><strong>To: </strong>bport, Plasma, romangg<br /><strong>Cc: </strong>romangg, davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>