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

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/121667/

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

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

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:

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,

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,

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

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

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

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

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 with the

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

Build failed in Jenkins: plasma-desktop_master_qt5 #903

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

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

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

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: 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 se...@kde.org: On Sunday,

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

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

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 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 in

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 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

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 thomas.pfeif...@kde.org 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

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 https://git.reviewboard.kde.org/r/121856/diff/1/?file=338402#file338402line44 General question: How's the QML API doc created? Can one inject a warning

Re: Minutes Monday Plasma Hangout

2015-01-05 Thread Marco Martin
On Jan 5, 2015 12:42 PM, Sebastian Kügler se...@kde.org 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

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 se...@kde.org wrote: Present: David, Kai Uwe, Martin G., Martin K., sebas Wasn't present (disabled by flu) but: get well soon *hugs* ___ Plasma-devel mailing

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 se...@kde.org wrote: Present: David, Kai Uwe, Martin G., Martin K., sebas Wasn't present (disabled by flu) but: get well soon *hugs*

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

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

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 thomas.pfeif...@kde.org 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

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line111 you're liable to break here. sometimes you call this directly not via the Qt

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line51 Use QScopedPointer Martin Klapetek wrote: What's wrong with properly used delete?

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line111 you're liable to break here. sometimes you call this directly not via the Qt

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line51 Use QScopedPointer Martin Klapetek wrote: What's wrong with properly used delete?

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 Gräßlin
On Jan. 3, 2015, 9:08 p.m., Kai Uwe Broulik wrote: applets/notifications/plugin/notificationshelper.cpp, line 51 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line51 Use QScopedPointer Martin Klapetek wrote: What's wrong with properly used delete?

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,

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 https://git.reviewboard.kde.org/r/121856/diff/1/?file=338402#file338402line44 General question: How's the QML API doc created? Can one inject a warning

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line111 you're liable to break here. sometimes you call this directly not via the Qt

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line111 you're liable to break here. sometimes you call this directly not via the Qt

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line111 you're liable to break here. sometimes you call this directly not via the Qt

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 https://git.reviewboard.kde.org/r/121360/diff/2/?file=337844#file337844line111 you're liable to break here. sometimes you call this directly not via the Qt

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