<table><tr><td style="">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/D14360">View Revision</a></tr></table><br /><div><div><p>Space is cheap. :) Verbosity is no problem as long as it communicates important information. How about this:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">* @returns whether the item's icon is editable, as some icons are not (e.g. the Trash has the capability to display two icons, representing full and empty states, and it's far simpler to make these icons non-editable than is it to provide an interface to edit them both)</pre></div></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/D14360">https://phabricator.kde.org/D14360</a></div></div><br /><div><strong>To: </strong>shubham, ngraham, broulik, Dolphin, Frameworks, pino<br /><strong>Cc: </strong>cfeck, pino, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>