Review Request 125972: Don't list systray containment actions in systray context menu.

Thomas Pfeiffer colomar at autistici.org
Mon Nov 9 23:36:12 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125972/#review88207
-----------------------------------------------------------


Why is that? How is the systray different from other widgets in the panel? And if we don't show panel options in the context menu of anything in the panel, how does a user get to them?

This is another issue that would be solved by a clear separation between a "usage mode" and a "config mode", btw.

- Thomas Pfeiffer


On Nov. 9, 2015, 10:43 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125972/
> -----------------------------------------------------------
> 
> (Updated Nov. 9, 2015, 10:43 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> as in "panel options" when systray is in the panel.
> 
> Only one of the formerly two (see https://git.reviewboard.kde.org/r/125973/) methods did this, so I think it's safe to remove from the one I'm keeping.
> 
> 
> Diffs
> -----
> 
>   applets/systemtray/plugin/protocols/plasmoid/plasmoidtask.cpp d378be14a4102144079b6a7819208e7bbd9b1bb7 
> 
> Diff: https://git.reviewboard.kde.org/r/125972/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151109/3dca4c81/attachment.html>


More information about the Plasma-devel mailing list