ndavis added a comment.
In D26238#583390 <https://phabricator.kde.org/D26238#583390>, @trmdi wrote: > In D26238#583389 <https://phabricator.kde.org/D26238#583389>, @ndavis wrote: > > > In the plasma panel context menu, add widgets and edit panel are below the separator > > > Maybe I need to create a new separator at the line 126 instead of reusing it. Is it ok if there are 2 separators in a context menu? Sounds reasonable, although I'm not sure they're unrelated enough that they need a separator. I'll leave it up to your judgement. > What about the other changes? They look fine. REPOSITORY R878 Latte Dock REVISION DETAIL https://phabricator.kde.org/D26238 To: trmdi, #latte_dock, #vdg Cc: ndavis, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
