[patch] QHttp and QFtp forbidden in KDE

2008-08-29 Thread Thiago Macieira
Those two classes are not allowed in KDE code. They won't follow the KDE settings, so using KIO is mandatory. The attached patch will accomplish this by causing any code using them to fail to compile. I will commit it next week. I know that the twitter plasmoid is using QHttp and will thus

Review Request: Autohide options for PanelController

2008-08-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/166/ --- Review request for Plasma. Summary --- this patch, really work in

Re: Review Request: Auto resize of fileWatcher plasmoid

2008-08-29 Thread michal seben
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/163/ --- (Updated 2008-08-29 07:37:19.114808) Review request for Plasma. Summary

Re: Review Request: Auto resize of fileWatcher plasmoid

2008-08-29 Thread michal seben
On 2008-08-27 16:24:38, Aaron Seigo wrote: there are a few issues to do with setting the size in init() for instance, but other than that (and a handful of style issues) it looks ok. the configuration dialog really needs some love though. hi Aaron thank for reply and time, I do

Re: [patch] PlasmaApp::adjustSize with multiple monitors

2008-08-29 Thread Guillaume Pothier
Aike, how can I test your work? I mean, I see it in playground and can probably compile it, but can you explain a bit what it does and how it works (from a user's point of view)? Or is that documented somewhere already? Thanks! g On Wed, Aug 27, 2008 at 8:23 AM, Aike J Sommer [EMAIL PROTECTED]

Re: [patch] PlasmaApp::adjustSize with multiple monitors

2008-08-29 Thread Guillaume Pothier
Aike, how can I test your work? I mean, I see it in playground and can probably compile it, but can you explain a bit what it does and how it works (from a user's point of view)? Or is that documented somewhere already? I talked too fast... I cannot compile it. It seems FindKephal.cmake is