<table><tr><td style="">hein requested changes to this revision.<br />hein 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/D23677">View Revision</a></tr></table><br /><div><div><p>Hmm, I'm not a fan. That only the isPopup case ends up showing the icon in the shell is something you and I know from the outside, but I don't think a widget should be making that assumption. Otherwise we bake implementation details of the containents into widget code, and if we ever change the former we have to fix the widgets. Take for example a widget list like the one bandied about for config modes on the sprint.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23677">https://phabricator.kde.org/D23677</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, hein<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>