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

Bhushan Shah bhush94 at gmail.com
Sun Jun 22 12:33:41 UTC 2014



> On June 22, 2014, 5:59 p.m., Eike Hein wrote:
> > Additionally, note that the "Place in system tray" option is also in frameworks/kio/src/widgets/kpropertiesdesktopadvbase.ui for the general .desktop app link properties UI (the one that also used to be loaded by the config dialog for Icon applets before we lost that due to the Qt Quick Controls switch).

Or maybe we can make this option do nothing for now.. and revisit this once someone ports ksystraycmd to sni? I opened this review because one of user at #plasma reported autostart apps not starting due to this thing.


- Bhushan


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


On June 22, 2014, 5:47 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, 5:47 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/20140622/aef1786b/attachment.html>


More information about the Plasma-devel mailing list