> 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
