ngraham added a comment.
In D27160#628867 <https://phabricator.kde.org/D27160#628867>, @ngraham wrote: > LGTM now. @broulik, are you good with this? > > There are some funky margins that I think are the fault of the pop-up itself, due to the recent header patch. Will investigate that. Should be fixed with D28089 <https://phabricator.kde.org/D28089>. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D27160 To: cblack, #vdg, #plasma, manueljlin, ngraham, broulik Cc: kmaterka, iasensio, ndavis, broulik, gvgeo, davidedmundson, ngraham, manueljlin, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart