Re: Review Request 121718: Allow to compile plasma-workspace without Qt5WebKit installed.

2015-01-05 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121718/ --- (Updated Jan. 6, 2015, 7:57 a.m.) Status -- This change has been mar

Re: testing a plasma applet with plasmoid viewer despite PackageUrlInterceptor

2015-01-05 Thread Philipp A.
well, you saw it: https://www.mail-archive.com/plasma-devel@kde.org/msg36985.html basically: file:///usr/lib/qt/qml/QtQuick/Controls/Styles/Plasma/FocusFrameStyle.qml: File not found file:///usr/lib/qt/qml/QtQuick/Controls/Styles/Base/ButtonStyle.qml:153:31: QML Item: Binding loop detected for pr

Re: Minutes Monday Plasma Hangout

2015-01-05 Thread Eike Hein
On 01/05/2015 03:46 PM, Sebastian Kügler wrote: Get better! Gute Besserung! Cheers, Eike ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread David Edmundson
> On Jan. 5, 2015, 3:19 p.m., David Edmundson wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 111 > > > > > > you're liable to break here. > > > > sometimes you call this directly n

Re: [kde-promo] Plasma naming scheme

2015-01-05 Thread Albert Astals Cid
El Dilluns, 5 de gener de 2015, a les 01:34:17, Aleix Pol va escriure: > On Sat, Jan 3, 2015 at 6:23 PM, Thomas Pfeiffer wrote: > > Hi everyone, > > while writing up a vision for Plasma interaction, the VDG noticed that it > > was unclear exactly what terms to use when referring to Plasma Desktop

Plasma NM 5.1.2.1

2015-01-05 Thread Jonathan Riddell
plasma-nm has gained an extra release, 5.1.2.1, which updates it to use networkmanager-qt (the new name of libnm-qt) 5.6.0 which is due to be released with the KDE Frameworks 5 release tomorrow. It will not compile with libnm-qt as released with Plasma 5.1.x, just keep using plasma-nm 5.1.2 is yo

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Klapetek
> On Jan. 5, 2015, 4:19 p.m., David Edmundson wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 111 > > > > > > you're liable to break here. > > > > sometimes you call this directly n

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Gräßlin
> On Jan. 5, 2015, 4:19 p.m., David Edmundson wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 111 > > > > > > you're liable to break here. > > > > sometimes you call this directly n

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Klapetek
> On Jan. 5, 2015, 4:19 p.m., David Edmundson wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 111 > > > > > > you're liable to break here. > > > > sometimes you call this directly n

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread David Edmundson
> On Jan. 5, 2015, 3:19 p.m., David Edmundson wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 111 > > > > > > you're liable to break here. > > > > sometimes you call this directly n

Re: Review Request 121856: KWindowSystem import for KDeclarative

2015-01-05 Thread David Edmundson
> On Jan. 5, 2015, 2:12 p.m., Thomas Lübking wrote: > > src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h, line 44 > > > > > > General question: > > How's the QML API doc created? > > Can one inject

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Klapetek
> On Jan. 5, 2015, 4:19 p.m., David Edmundson wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 111 > > > > > > you're liable to break here. > > > > sometimes you call this directly n

Re: Review Request 121718: Allow to compile plasma-workspace without Qt5WebKit installed.

2015-01-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121718/#review73174 --- Ship it! Ship It! - Aleix Pol Gonzalez On Jan. 5, 2015, 8:

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Gräßlin
> On Jan. 3, 2015, 9:08 p.m., Kai Uwe Broulik wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 51 > > > > > > Use QScopedPointer > > Martin Klapetek wrote: > What's wrong with properly us

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121360/#review73170 --- applets/notifications/plugin/notificationshelper.cpp

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Klapetek
> On Jan. 3, 2015, 9:08 p.m., Kai Uwe Broulik wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 51 > > > > > > Use QScopedPointer > > Martin Klapetek wrote: > What's wrong with properly us

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread David Edmundson
> On Jan. 3, 2015, 8:08 p.m., Kai Uwe Broulik wrote: > > applets/notifications/plugin/notificationshelper.cpp, line 51 > > > > > > Use QScopedPointer > > Martin Klapetek wrote: > What's wrong with properly us

Re: Minutes Monday Plasma Hangout

2015-01-05 Thread Sebastian Kügler
On Monday, January 05, 2015 15:45:29 Martin Gräßlin wrote: > Am 2015-01-05 15:36, schrieb Marco Martin: > > On Jan 5, 2015 12:42 PM, "Sebastian Kügler" wrote: > > > > > > Present: David, Kai Uwe, Martin G., Martin K., sebas > > > > > > > > Wasn't present (disabled by flu) but: > get well soon *

Re: Minutes Monday Plasma Hangout

2015-01-05 Thread Martin Gräßlin
Am 2015-01-05 15:36, schrieb Marco Martin: On Jan 5, 2015 12:42 PM, "Sebastian Kügler" wrote: > > Present: David, Kai Uwe, Martin G., Martin K., sebas Wasn't present (disabled by flu) but: get well soon *hugs* ___ Plasma-devel mailing list Plasma-

Re: Minutes Monday Plasma Hangout

2015-01-05 Thread Marco Martin
On Jan 5, 2015 12:42 PM, "Sebastian Kügler" wrote: > > Present: David, Kai Uwe, Martin G., Martin K., sebas Wasn't present (disabled by flu) but: Worked with david to get kpackage working without regressions on old workspace Worked on the plotter patch, but in the end didn't merge for this releas

Re: Review Request 121856: KWindowSystem import for KDeclarative

2015-01-05 Thread Martin Gräßlin
> On Jan. 5, 2015, 3:12 p.m., Thomas Lübking wrote: > > src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h, line 44 > > > > > > General question: > > How's the QML API doc created? > > Can one inject

Re: Review Request 121856: KWindowSystem import for KDeclarative

2015-01-05 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121856/#review73164 --- src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121360/ --- (Updated Jan. 5, 2015, 2:33 p.m.) Review request for Plasma and Kai Uwe B

Re: Plasma naming scheme

2015-01-05 Thread Jens Reuterberg
On Monday 05 January 2015 01:34:17 Aleix Pol wrote: > On Sat, Jan 3, 2015 at 6:23 PM, Thomas Pfeiffer wrote: > > Hi everyone, > > while writing up a vision for Plasma interaction, the VDG noticed that it > > was unclear exactly what terms to use when referring to Plasma Desktop > > specifically

Re: Review Request 121856: KWindowSystem import for KDeclarative

2015-01-05 Thread Martin Gräßlin
> On Jan. 5, 2015, 1:25 p.m., Martin Gräßlin wrote: > > why are the methods static? > > > > Concerning compositing: wait some time after triggering the shortcut. KWin > > only updates after some time to not create too much load if it's just going > > to be restarted. > > Kai Uwe Broulik wrote

Re: Review Request 121856: KWindowSystem import for KDeclarative

2015-01-05 Thread Kai Uwe Broulik
> On Jan. 5, 2015, 12:25 nachm., Martin Gräßlin wrote: > > why are the methods static? > > > > Concerning compositing: wait some time after triggering the shortcut. KWin > > only updates after some time to not create too much load if it's just going > > to be restarted. Because they're static

Re: Review Request 121856: KWindowSystem import for KDeclarative

2015-01-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121856/#review73142 --- why are the methods static? Concerning compositing: wait some

Re: Review Request 121850: Port KRunner to categorized logging

2015-01-05 Thread Luca Beltrame
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121850/ --- (Updated Jan. 5, 2015, 12:20 p.m.) Status -- This change has been ma

Re: testing a plasma applet with plasmoid viewer despite PackageUrlInterceptor

2015-01-05 Thread Sebastian Kügler
On Monday, January 05, 2015 13:01:41 Philipp A. wrote: > doesn’t change the fact that i see nothing but a blank plasma applet > container. (only the border, nothing in it) No errors or warnings hidden in the slurry of output? > 2015-01-05 11:13 GMT+01:00 Sebastian Kügler : > > On Sunday, December

Re: testing a plasma applet with plasmoid viewer despite PackageUrlInterceptor

2015-01-05 Thread Philipp A.
doesn’t change the fact that i see nothing but a blank plasma applet container. (only the border, nothing in it) 2015-01-05 11:13 GMT+01:00 Sebastian Kügler : > On Sunday, December 28, 2014 21:21:12 Philipp A. wrote: > > when using a custom interceptor to circumvent it, i still get strange > > er

Re: Review Request 121850: Port KRunner to categorized logging

2015-01-05 Thread Milian Wolff
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121850/#review73136 --- Ship it! src/declarative/krunner_debug.h

Minutes Monday Plasma Hangout

2015-01-05 Thread Sebastian Kügler
Present: David, Kai Uwe, Martin G., Martin K., sebas 5th January, 2014 David - worked with Marco on getting KPackage merged flawlessly - Black screen bug triggered by KScreen ended up being in KActivities [fixed] - Baloo as framework discussion going on - Will transition to Wayland work shortly -

Re: Review Request 121360: Rework Plasma's notification positioning code

2015-01-05 Thread Martin Klapetek
> On Jan. 3, 2015, 9:08 p.m., Kai Uwe Broulik wrote: > > Overall the notification appearance is much cleaner, they don't overlap > > anymore and when they're updated they don't just flicker re-appear but fade > > out and fade in again. > > However, the vertical positioning is completely off wit

Review Request 121856: KWindowSystem import for KDeclarative

2015-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121856/ --- Review request for kwin and Plasma. Repository: kdeclarative Descriptio

Re: Review Request 121806: Allow desktop shell to force a window active

2015-01-05 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121806/ --- (Updated Jan. 5, 2015, 10:42 vorm.) Status -- This change has been d

Re: Review Request 121738: Let kcmkded also handle kded modules that use JSON metadata

2015-01-05 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121738/#review73127 --- Ship it! Ship It! - Sebastian Kügler On Dec. 29, 2014, 6:0

Re: testing a plasma applet with plasmoid viewer despite PackageUrlInterceptor

2015-01-05 Thread Sebastian Kügler
On Sunday, December 28, 2014 21:21:12 Philipp A. wrote: > when using a custom interceptor to circumvent it, i still get strange > errors. can someone help? > > QML debugging is enabled. Only use this in a safe environment. > Constructing a KPluginInfo object from old style JSON. Please use > kcore

Re: Review Request 119814: [ksld] ScreenLocker inhibits sleep on logind

2015-01-05 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119814/ --- (Updated Jan. 5, 2015, 10:46 a.m.) Review request for Plasma and Àlex Fie

Re: Review Request 121850: Port KRunner to categorized logging

2015-01-05 Thread Luca Beltrame
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121850/ --- (Updated Gen. 5, 2015, 9:45 a.m.) Review request for KDE Frameworks, Plas

Re: Review Request 121850: Port KRunner to categorized logging

2015-01-05 Thread Luca Beltrame
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121850/ --- (Updated Gen. 5, 2015, 9:41 a.m.) Review request for KDE Frameworks, Plas

Review Request 121850: Port KRunner to categorized logging

2015-01-05 Thread Luca Beltrame
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121850/ --- Review request for KDE Frameworks, Plasma and Vishesh Handa. Bugs: 342491

Jenkins build is back to normal : plasma-desktop_master_qt5 #904

2015-01-05 Thread KDE CI System
See ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Build failed in Jenkins: plasma-desktop_master_qt5 #903

2015-01-05 Thread KDE CI System
See Changes: [faure] Port foldermodel from KonqOperations::doDrop to the new KIO::drop() job [faure] Remove now-unused KonqOperations copy. -- [...truncated 1505 lines...] ^

Re: Review Request 121718: Allow to compile plasma-workspace without Qt5WebKit installed.

2015-01-05 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121718/ --- (Updated Jan. 5, 2015, 8:24 a.m.) Review request for Plasma. Changes --

Re: Review Request 121476: Port showdesktop applet to Plasma Next

2015-01-05 Thread Ashish Madeti
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121476/ --- (Updated Jan. 5, 2015, 8:14 a.m.) Status -- This change has been mar

Re: Review Request 121476: Port showdesktop applet to Plasma Next

2015-01-05 Thread Martin Gräßlin
> On Jan. 2, 2015, 10:25 a.m., Martin Gräßlin wrote: > > The adjustments look good to me. If we want it for the next plasma release > > I call it a "ShipIt". If we want to delay by one release the code could be > > extremely simplified by making use of > > https://git.reviewboard.kde.org/r/121

Re: Review Request 119814: [ksld] ScreenLocker inhibits sleep on logind

2015-01-05 Thread Martin Gräßlin
> On Jan. 4, 2015, 6:05 p.m., Kai Uwe Broulik wrote: > > One issue I found while running this patch is that when the session resumes > > the screen is blank until I move the mouse or press a key. PowerDevil calls > > simulateUserActivity but that doesn't seem to work anymore with your recent >