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

Sebastian Kügler sebas at kde.org
Fri Nov 7 14:39:18 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?
> 
> Aleix Pol Gonzalez wrote:
>     Well, what would you get instead? You still gets the window decoration anyway.

Brainfart on my end of the line. I *thought* this was for the actual notifier icons (which of course makes uhm ... very limited sense ;)). theme.backgroundColor is just fine there.


- Sebastian


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


On Nov. 7, 2014, 2:08 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. 7, 2014, 2:08 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/plasma-devel/attachments/20141107/f4c0e5b4/attachment.html>


More information about the Plasma-devel mailing list