Review Request 118876: Remove start in systemtray option from the kmenuedit.

[Sebastian Kügler]

> On June 22, 2014, 6:58 p.m., Martin Gräßlin wrote:
> > I think the suggestion to port ksystemtraycmd to SNI is wrong: we don't want applications in the system tray. Removing X11 support from system tray is one part of the story for that.

We do want to allow apps in the systemtray in principle, but be able to move them elsewhere. So a ksystemtraycmd using SNI would accomplish that (since it attaches enough metadata, category for example). Not supporting X11/FDO systray spec is a technical thing (it's almost impossible to do well in a scenegraph world). Therefore, porting ksystemtraycmd to SNI makes sense.


- Sebastian


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


On June 22, 2014, 12:36 p.m., Bhushan Shah wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118876/
> -----------------------------------------------------------
> 
> (Updated June 22, 2014, 12:36 p.m.)
> 
> 
> Review request for Plasma and Àlex Fiestas.
> 
> 
> Repository: kmenuedit
> 
> 
> Description
> -------
> 
> This option uses ksystraycmd tool which is removed long ago
> 
> http://commits.kde.org/kde-workspace/38fddebe00a271436c17120a0adffeb0f52a1af7
> 
> 
> Diffs
> -----
> 
>   basictab.h 9aa4c18 
>   basictab.cpp 6e3e086 
> 
> Diff: https://git.reviewboard.kde.org/r/118876/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bhushan Shah
> 
>

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