<table><tr><td style="">davidre 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/D27848">View Revision</a></tr></table><br /><div><div><p>I think this was done at the wrong level. If we don't want icons in the systray  then the way would be to net set the property in the systray and not ignore the property in the generic component but keep the property for api stability purposes. Now we have a component with a documented property that doesn't work. People trying to use this will think it's broken while we can't use it in any context where we want an icon in the future. Furthermore it's inconsistent that one can still set an icon via image strengthening the broken look from the outside.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27848">https://phabricator.kde.org/D27848</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma, cblack, niccolove, apol<br /><strong>Cc: </strong>davidre, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>