<table><tr><td style="">romangg accepted this revision.<br />romangg added a comment.<br />This revision is now accepted and ready to land.
</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/D9255" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I'll give this an ok, since the base functionality is there. I'm not sure if I understood the reasoning completely why the context menu can't be aligned correctly to the context menu button (something asynchronously, but the final position of the menu button is known at some point for sure), but maybe this can be fixed later. Same for the graphical issue with stretched lines.</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/D9255" rel="noreferrer">https://phabricator.kde.org/D9255</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG, ngraham, romangg<br /><strong>Cc: </strong>januz, romangg, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>