D26238: Improve context menu
Noah Davis
noreply at phabricator.kde.org
Fri Dec 27 05:17:12 GMT 2019
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191227/7c3d3712/attachment.html>
More information about the Plasma-devel
mailing list