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