[Powerdevil] [Bug 353538] screen switches off when playing video in VLC or even when screen blanking is switched off

2016-04-29 Thread Martin Steigerwald via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=353538 Martin Steigerwald changed: What|Removed |Added Resolution|WAITINGFORINFO |WORKSFORME

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 63 - Failure!

2016-04-29 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/63/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 29 Apr 2016 16:35:46 + Build duration: 9 min 25 sec CHANGE SET Revision

Re: Review Request 127793: kuiserver: use QApplication rather than QCoreApplication

2016-04-29 Thread David Edmundson
> On April 29, 2016, 4:27 p.m., David Edmundson wrote: > > Which part of ProgressListModel uses a UI? > > Rex Dieter wrote: > Good question, I was largely just parot'ing comments from bug #348123 > about this proposed patch, I'll go digging. > > Rex Dieter wrote: > Further detail from

Re: Review Request 127793: kuiserver: use QApplication rather than QCoreApplication

2016-04-29 Thread Rex Dieter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127793/ --- (Updated April 29, 2016, 4:34 p.m.) Status -- This change has been

Re: Review Request 127793: kuiserver: use QApplication rather than QCoreApplication

2016-04-29 Thread Rex Dieter
> On April 29, 2016, 4:27 p.m., David Edmundson wrote: > > Which part of ProgressListModel uses a UI? > > Rex Dieter wrote: > Good question, I was largely just parot'ing comments from bug #348123 > about this proposed patch, I'll go digging. Further detail from the bug in question in case

Re: Review Request 127793: kuiserver: use QApplication rather than QCoreApplication

2016-04-29 Thread Rex Dieter
> On April 29, 2016, 4:27 p.m., David Edmundson wrote: > > Which part of ProgressListModel uses a UI? Good question, I was largely just parot'ing comments from bug #348123 about this proposed patch, I'll go digging. - Rex --- This is

Re: Review Request 127793: kuiserver: use QApplication rather than QCoreApplication

2016-04-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127793/#review95026 --- Ship it! Which part of ProgressListModel uses a UI? -

Review Request 127793: kuiserver: use QApplication rather than QCoreApplication

2016-04-29 Thread Rex Dieter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127793/ --- Review request for Plasma. Bugs: 348123

Review Request 127790: Fix usage of string "shut down"

2016-04-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127790/ --- Review request for Plasma. Repository: plasma-desktop Description

Review Request 127791: Fix usage of string "shut down"

2016-04-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127791/ --- Review request for Plasma. Repository: plasma-workspace Description

[frameworks-kactivities-stats] [Bug 362198] Plasma Crashes

2016-04-29 Thread David Edmundson via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=362198 David Edmundson changed: What|Removed |Added Component|general |general

Re: Considering a Qt 5.6 dependency for plasma-workspace

2016-04-29 Thread Bhushan Shah
On Fri, Apr 29, 2016 at 8:15 PM, Kai Uwe Broulik wrote: > Is there an eta for Qt 5.6 on Neon? I'm not really into building Qt from git > and use that. Yes, Neon unstable already uses Qt 5.6 and so is mobile using now -- Bhushan Shah http://bhush9.github.io IRC Nick :

Re: Considering a Qt 5.6 dependency for plasma-workspace

2016-04-29 Thread Kai Uwe Broulik
Is there an eta for Qt 5.6 on Neon? I'm not really into building Qt from git and use that. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Considering a Qt 5.6 dependency for plasma-workspace

2016-04-29 Thread Aleix Pol
On Thu, Mar 31, 2016 at 1:54 PM, Sebastian Kügler wrote: > On Friday, March 25, 2016 04:25:54 PM Martin Graesslin wrote: >> On Thursday, March 24, 2016 5:31:21 PM CET Aleix Pol wrote: >> > Hi, >> > I'd like to suggest a Qt 5.6 dependency for plasma-workspace master >> > (and maybe

Re: Review Request 125451: RFC: Drop KScreen dependency from PlasmaShell

2016-04-29 Thread Sebastian Kügler
> On April 29, 2016, 2:06 p.m., David Edmundson wrote: > > lets get this merged ASAP so we have some time to fix any fallout from it. > > Sebastian Kügler wrote: > But we don't depend on Qt 5.6 yet in workspace. I think the blocker there > is that we need Qt 5.6 on the CI system, then we

Re: Review Request 125451: RFC: Drop KScreen dependency from PlasmaShell

2016-04-29 Thread David Edmundson
> On April 29, 2016, 2:06 p.m., David Edmundson wrote: > > lets get this merged ASAP so we have some time to fix any fallout from it. > > Sebastian Kügler wrote: > But we don't depend on Qt 5.6 yet in workspace. I think the blocker there > is that we need Qt 5.6 on the CI system, then we

Re: Review Request 125451: RFC: Drop KScreen dependency from PlasmaShell

2016-04-29 Thread Sebastian Kügler
> On April 29, 2016, 2:06 p.m., David Edmundson wrote: > > lets get this merged ASAP so we have some time to fix any fallout from it. But we don't depend on Qt 5.6 yet in workspace. I think the blocker there is that we need Qt 5.6 on the CI system, then we can switch. Incidentally, Qt 5.6 on

Re: Review Request 125451: RFC: Drop KScreen dependency from PlasmaShell

2016-04-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125451/#review95021 --- lets get this merged ASAP so we have some time to fix any

Re: Plasma appet with C++ plugin

2016-04-29 Thread Ashish Bansal
On Fri, Apr 29, 2016 at 3:09 PM, Marco Martin wrote: > On Tuesday 26 April 2016 22:03:11 Pablo Sánchez Rodríguez wrote: > > Hello, > > > > My name is Pablo and currently I am writing an applet for Plasma > (Plasmoid). > > > > Due to new functions I would like to implement in

Re: Review Request 125451: RFC: Drop KScreen dependency from PlasmaShell

2016-04-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125451/#review95020 --- Ship it! Ship It! - David Edmundson On March 17, 2016,

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 62 - Still Unstable!

2016-04-29 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/62/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 29 Apr 2016 13:21:06 + Build duration: 9 min 53 sec CHANGE SET Revision

Re: Review Request 127789: Media Controller: Fix playing next/previous song seeking

2016-04-29 Thread Rajeesh K V
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127789/ --- (Updated April 29, 2016, 6:20 a.m.) Status -- This change has been

Re: Review Request 127789: Media Controller: Fix playing next/previous song seeking

2016-04-29 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127789/#review95018 --- Fix it, then Ship it! Lgtm, thanks It probably still

[Differential] [Request, 4 lines] D1509: Honor input mask set on internal windows

2016-04-29 Thread Martin Gräßlin
graesslin created this revision. graesslin added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This ensures that QWindow::setMask works for KWin internal windows. Without KWin sends all pointer

Review Request 127789: Media Controller: Fix playing next/previous song seeking

2016-04-29 Thread Rajeesh K V
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127789/ --- Review request for Plasma and Kai Uwe Broulik. Bugs: 362473

Re: Review Request 127780: Don't open kdeglobals explicitly in font/hint settings.

2016-04-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127780/ --- (Updated April 29, 2016, 11:30 a.m.) Status -- This change has been

Re: Review Request 127780: Don't open kdeglobals explicitly in font/hint settings.

2016-04-29 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127780/#review95015 --- Still lgtm. src/platformtheme/kfontsettingsdata.cpp (line

Re: Review Request 127780: Don't open kdeglobals explicitly in font/hint settings.

2016-04-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127780/ --- (Updated April 29, 2016, 11:23 a.m.) Review request for Plasma.

Re: Review Request 127755: [IconItem] Clearing pixmap makes toolbuttons invisible in some conditions

2016-04-29 Thread Anthony Fieroni
> On Април 29, 2016, 12:49 след обяд, Marco Martin wrote: > > you removed the way it had to disable animations when it passed from > > visible to invisible. > > this will probably bring back the weird animation of icons when the > > tooltips come up I can't notice any icon animation whe

Re: Review Request 127702: ConfigModel: Don't try to resolve empty source path from package

2016-04-29 Thread David Rosca
> On April 29, 2016, 9:51 a.m., Marco Martin wrote: > > any news on this? Sorry, not much time now. But I did take a look and the test will need AppletInterface to actually test this case, so I think it won't be easy from plasma-framework. With the plasmashell testing subsystem added

Re: Review Request 127702: ConfigModel: Don't try to resolve empty source path from package

2016-04-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127702/#review95007 --- any news on this? - Marco Martin On April 21, 2016, 10:11

Re: Review Request 127755: [IconItem] Clearing pixmap makes toolbuttons invisible in some conditions

2016-04-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127755/#review95006 --- you removed the way it had to disable animations when it

Re: Review Request 125648: Add method to run a script from DBus directly

2016-04-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review95005 --- Ship it! last change i would like is to not run the

Re: Plasma appet with C++ plugin

2016-04-29 Thread Marco Martin
On Tuesday 26 April 2016 22:03:11 Pablo Sánchez Rodríguez wrote: > Hello, > > My name is Pablo and currently I am writing an applet for Plasma (Plasmoid). > > Due to new functions I would like to implement in my applet, I need to > create a plugin written in C++ (using Kdeclarative). > > My

Re: Feedback on xdg-shell v5

2016-04-29 Thread Pekka Paalanen
On Fri, 29 Apr 2016 15:31:28 +0800 Jonas Ådahl wrote: > On Thu, Apr 28, 2016 at 02:08:07PM +0300, Pekka Paalanen wrote: > > On Thu, 28 Apr 2016 17:36:58 +0800 > > Jonas Ådahl wrote: > > > > > On Thu, Apr 28, 2016 at 10:36:14AM +0200, Martin Graesslin

[Differential] [Accepted] D1487: Don't depend on operation mode in WindowPixmap

2016-04-29 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. mart added a comment. This revision is now accepted and ready to land. even in my unexpertness of kwin, it makes sense to me tough REPOSITORY rKWIN KWin BRANCH window-pixmap-improve-checks REVISION DETAIL

Re: Feedback on xdg-shell v5

2016-04-29 Thread Pekka Paalanen
On Thu, 28 Apr 2016 17:36:58 +0800 Jonas Ådahl wrote: > On Thu, Apr 28, 2016 at 10:36:14AM +0200, Martin Graesslin wrote: > > Hi Wayland and Plasma, > > > > I finished the implementation of xdg-shell in KWayland [1] and KWin [2]. > > Overall it was more straight forward than

[Differential] [Accepted] D1481: Make integration with KScreenLocker optional in WaylandServer

2016-04-29 Thread bshah (Bhushan Shah)
bshah accepted this revision. bshah added a reviewer: bshah. This revision is now accepted and ready to land. REPOSITORY rKWIN KWin BRANCH disable-lockscreen-integration REVISION DETAIL https://phabricator.kde.org/D1481 EMAIL PREFERENCES

[Differential] [Request, 2 lines] D1486: Only create damage_handle for Toplevel's without a Wayland Surface

2016-04-29 Thread Martin Gräßlin
graesslin created this revision. graesslin added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY If the Toplevel has a Wayland Surface we don't want an X11 damage handle. Ideally we would just check

[Differential] [Request, 79 lines] D1481: Make integration with KScreenLocker optional in WaylandServer

2016-04-29 Thread Martin Gräßlin
graesslin created this revision. graesslin added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY In order to start the WaylandServer in kwin_x11 we need to make sure that WaylandServer does not start

Re: Feedback on xdg-shell v5

2016-04-29 Thread Jonas Ådahl
On Fri, Apr 29, 2016 at 10:37:49AM +0200, Martin Graesslin wrote: > On Thursday, April 28, 2016 5:36:58 PM CEST Jonas Ådahl wrote: > > On Thu, Apr 28, 2016 at 10:36:14AM +0200, Martin Graesslin wrote: > > > Hi Wayland and Plasma, > > > > > > I finished the implementation of xdg-shell in KWayland

Re: Feedback on xdg-shell v5

2016-04-29 Thread Jonas Ådahl
On Thu, Apr 28, 2016 at 10:36:14AM +0200, Martin Graesslin wrote: > Hi Wayland and Plasma, > > I finished the implementation of xdg-shell in KWayland [1] and KWin [2]. > Overall it was more straight forward than I would have expected. Especially > the changes to KWin were surprisingly minor

Re: Feedback on xdg-shell v5

2016-04-29 Thread Jonas Ådahl
On Thu, Apr 28, 2016 at 02:08:07PM +0300, Pekka Paalanen wrote: > On Thu, 28 Apr 2016 17:36:58 +0800 > Jonas Ådahl wrote: > > > On Thu, Apr 28, 2016 at 10:36:14AM +0200, Martin Graesslin wrote: > > > Hi Wayland and Plasma, > > > > > > I finished the implementation of xdg-shell

Re: Plasma & Qt 5.7

2016-04-29 Thread Takahiro HASHIMOTO
HI, I also tried a few days ago. This is the just a quick response :) It is due to this change. [1] I think. Here is my ugly patch to rework libdbusmenu-qt. I think that the better solution is refine QDBusMenuShortcut, not inheriting QList... [1] https://codereview.qt-project.org/#/c/144823/

Plasma appet with C++ plugin

2016-04-29 Thread Pablo Sánchez Rodríguez
Hello, My name is Pablo and currently I am writing an applet for Plasma (Plasmoid). Due to new functions I would like to implement in my applet, I need to create a plugin written in C++ (using Kdeclarative). My questions are: Does anybody know where I can find documentation about how to do

[Differential] [Updated, 19 lines] D1501: [autotest] Work around "cannot assign undefined" warning

2016-04-29 Thread Martin Gräßlin
graesslin updated this revision to Diff 3536. graesslin added a comment. Add back code deleted by accident REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1501?vs=3535=3536 BRANCH autotest-memory-management

[Differential] [Accepted] D1501: [autotest] Work around "cannot assign undefined" warning

2016-04-29 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma BRANCH autotest-memory-management REVISION DETAIL https://phabricator.kde.org/D1501 EMAIL PREFERENCES

[Differential] [Accepted] D1455: Auto-test to verify that the empty defaults are loaded correctly in KHintsSettings

2016-04-29 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma BRANCH hintsettings-autotest-default REVISION DETAIL https://phabricator.kde.org/D1455 EMAIL PREFERENCES

Re: Review Request 127779: use system colors for monochrome icons

2016-04-29 Thread Marco Martin
> On April 28, 2016, 5:36 p.m., Aleix Pol Gonzalez wrote: > > src/kiconloader.cpp, line 861 > > > > > > We will need that and expose it properly, as we'll have to request the > > icon cache to clean if it's one

Re: Review Request 127779: use system colors for monochrome icons

2016-04-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127779/ --- (Updated April 29, 2016, 9:25 a.m.) Review request for KDE Frameworks

Re: Feedback on xdg-shell v5

2016-04-29 Thread Martin Graesslin
On Thursday, April 28, 2016 2:08:07 PM CEST Pekka Paalanen wrote: > > > The biggest problem for me is the request set_window_geometry. I think I > > > mentioned it already before on the wayland list. Basically I have no > > > idea how to implement that in KWin. We have only one geometry for a > >

Re: Feedback on xdg-shell v5

2016-04-29 Thread Martin Graesslin
On Thursday, April 28, 2016 5:36:58 PM CEST Jonas Ådahl wrote: > On Thu, Apr 28, 2016 at 10:36:14AM +0200, Martin Graesslin wrote: > > Hi Wayland and Plasma, > > > > I finished the implementation of xdg-shell in KWayland [1] and KWin [2]. > > Overall it was more straight forward than I would have

Re: Review Request 127780: Don't open kdeglobals explicitly in font/hint settings.

2016-04-29 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127780/#review95003 --- src/platformtheme/khintssettings.cpp (line 58)