<table><tr><td style="">mart requested changes to this revision.<br />mart 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/D24567">View Revision</a></tr></table><br /><div><div><p>This is more a comment on the technical side.<br />
It seems quite a brittle implementation to me, and quite prone to breakage (anything that relies on PlasmaCore.Dialog trying to make with it things that weren't in its architecture is bound to break, and that class in general is not much touchable.<br />
also, assumes that the systray will be positioned in a panelon a screen edge, and since this is not enforced, it should not be assumed.</p>

<p>Anything that wants to do such a sidebar.. that's really a panel (different window flags, different many things, but still is a containment of Plasma::Types::Vertical form factor and Plasma::Types::Right location, withthe plasmoids inside that would react to that concept.</p>

<p>That would be quite an huge things touching pretty much every level down to plasma-framework, but i feel this approach wouldn't really work.</p>

<p>On this systray plasmoid only, I would already prefer trying to give it a resize handle and allow generic vertical resizing (so also allowing to resize it to full height) is still not very robust, but at least kinda makes sense when the systray is not exactly where is expected (if necessary, there is quite some popup resizing code that was deleted still buried somewhere in years old git history)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24567">https://phabricator.kde.org/D24567</a></div></div><br /><div><strong>To: </strong>filipf, VDG, Plasma, mart<br /><strong>Cc: </strong>mart, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra<br /></div>