Tooltips to QML: PlasmaCore.Dialog questions

2012-06-03 Thread Mark
Hi, Since "some" people seem to blame me a little for asking question in private mails.. I will throw a bunch of questions in the list. For starters, my tooltip "endeavors" are really starting to show now. I got the tooltip somewhat working with nice red text in it and a nice red square for the i

Re: Bugsplashing

2012-06-03 Thread Marco Martin
On Saturday 02 June 2012, Thijs Heus wrote: > Hello Plasma-devvers, > > Myriams actions at bugs.kde.org stirred my email box last week, and > actually got me to do plasma bug triaging myself again. It is awesome to > see that the number of bug reports for plasma is now less than half of what > it

Re: Review Request: Plasmate: request to merge terietor/remoteinstaller to master

2012-06-03 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105032/#review14399 --- publisher/remoteinstaller/remoteinstaller.cpp

Re: Review Request: Plasmate: request to merge terietor/remoteinstaller to master

2012-06-03 Thread David Edmundson
> On June 2, 2012, 3:14 p.m., David Edmundson wrote: > > publisher/remoteinstaller/remoteinstallerdialog.cpp, line 53 > > > > > > This looks like it leaks. > > > > QScopedPointer perhaps? > > Giorgos Tsiapa

Survey to prepare 'Next Workspace Iteration' sprint

2012-06-03 Thread Björn Balazs
Hi all, I have prepared a survey to gather wide input for the Sprint next week. I want to start the survey tomorrow evening (Monday, CEST), and I would like to get some feedback from you guys untill then. THE SURVEY DOES NOT YET RECORD ANYTHING (So, please do not spread the link untill I say

Re: Review Request: Plasmate: Add tabbox previewer

2012-06-03 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105139/ --- (Updated June 3, 2012, 4:21 p.m.) Review request for kwin, Plasma and Mart

Re: Review Request: Plasmate: Add tabbox previewer

2012-06-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105139/#review14395 --- I have not yet tried it. From the code it looks quite ok, only

Re: Review Request: Plasmate: Add tabbox previewer

2012-06-03 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105139/ --- (Updated June 3, 2012, 10:01 a.m.) Review request for kwin, Plasma and Mar

Review Request: Plasmate: Add tabbox previewer

2012-06-03 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105139/ --- Review request for kwin, Plasma and Martin Gräßlin. Description --- H

Re: Review Request: Plasmate: request to merge terietor/remoteinstaller to master

2012-06-03 Thread Giorgos Tsiapaliwkas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105032/ --- (Updated June 3, 2012, 9:32 a.m.) Review request for Plasma. Changes ---

Re: Review Request: Plasmate: request to merge terietor/remoteinstaller to master

2012-06-03 Thread Giorgos Tsiapaliwkas
> On June 2, 2012, 3:14 p.m., David Edmundson wrote: > > publisher/remoteinstaller/remoteinstallerdialog.cpp, line 80 > > > > > > Why is this needed? When you create m_installer, you pass this as the > > parent. >

Re: Review Request: Plasmate: request to merge terietor/remoteinstaller to master

2012-06-03 Thread Giorgos Tsiapaliwkas
> On June 1, 2012, 6:40 p.m., Aaron J. Seigo wrote: > > publisher/remoteinstaller/remoteinstaller.cpp, line 88 > > > > > > this is the command that fails when there is no display env var set? > > > > if so .

Re: Review Request: Plasmate: request to merge terietor/remoteinstaller to master

2012-06-03 Thread Giorgos Tsiapaliwkas
> On June 2, 2012, 3:14 p.m., David Edmundson wrote: > > publisher/remoteinstaller/remoteinstaller.cpp, line 54 > > > > > > Is this display hack needed with the newly patched plasmapkg? since there is no need for th