Review Request 121016: [RFC] Make it possible to make plasmawindow live in a status notifier

Aleix Pol Gonzalez aleixpol at kde.org
Fri Nov 7 14:04:42 UTC 2014



> On Nov. 7, 2014, 11:20 a.m., Sebastian Kügler wrote:
> > plasma-windowed/plasmawindowedview.cpp, line 44
> > <https://git.reviewboard.kde.org/r/121016/diff/1/?file=326609#file326609line44>
> >
> >     I wonder if the background here should be transparant?

Well, what would you get instead? You still gets the window decoration anyway.


> On Nov. 7, 2014, 11:20 a.m., Sebastian Kügler wrote:
> > plasma-windowed/main.cpp, line 46
> > <https://git.reviewboard.kde.org/r/121016/diff/1/?file=326605#file326605line46>
> >
> >     --statusnotifier perhaps? That would be consistent with the naming in the code.

Ok will change it. I left systray because I thought people would understand it better, but oh well, whoever uses this flag should know what a status notifier as well.


- Aleix


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


On Nov. 6, 2014, 6:55 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121016/
> -----------------------------------------------------------
> 
> (Updated Nov. 6, 2014, 6:55 p.m.)
> 
> 
> Review request for kdeconnect and Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> [background explaination]
> I was thinking yesterday about the problem KDE Connect has that it's not really usable in any non-Plasma workspace because the main interface is the plasmoid.
> I understand that this won't solve all problems, but it can be a first approach for making plasmoids less of a niche solution, making it possible to let the solutions we have currently available elsewhere (it's especially interesting considering that most of our useful plasmoids live on the panel).
> The end goal is that KDE Connect can provide a desktop file with NotShowIn=KDE that lauches the plasmoid in a status notifier.
> [End of background explaination]
> 
> Now this introduces a --systray argument (can be changed, I couldn't find a better name) that sets up a status notifier item and when the window is closed it doesn't kill the process. It's quite straightforward, but needs polishing I guess.
> 
> 
> Diffs
> -----
> 
>   plasma-windowed/CMakeLists.txt 252724a 
>   plasma-windowed/main.cpp 649bd39 
>   plasma-windowed/plasmawindowedcorona.h 8e6fd76 
>   plasma-windowed/plasmawindowedcorona.cpp 849fa51 
>   plasma-windowed/plasmawindowedview.h a6a8a44 
>   plasma-windowed/plasmawindowedview.cpp 4e0ac65 
> 
> Diff: https://git.reviewboard.kde.org/r/121016/diff/
> 
> 
> Testing
> -------
> 
> Played around with it for a while.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20141107/fa480eff/attachment.html>


More information about the KDEConnect mailing list