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

2016-04-29 Thread Kai Uwe Broulik via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=353538 --- Comment #3 from Kai Uwe Broulik --- Thanks for reporting back. However if battery monitor reports that vlc blocks power management it shouldn't dim the screen and here this also works fine, especially since disabling pm globally works. -- You are

[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 Status|NEEDSINFO

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

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 m

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 a

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

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 https://bugs.kde.org/show_bug

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 Assignee|k...@davidedmundso

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 : bshah on Freenode ___

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

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 ca

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 ca

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 P

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 fal

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 my applet, I need to

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

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 m

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 send

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

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 https://b

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 Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127780/#review95016 --- Ship it! Ship It! - Martin Gräßlin On April 29, 2016, 1

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 4

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

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 toolti

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 recently

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 pass

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 script

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 ques

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 wrote: > > > > Hi Wayland and Plasma, >

[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 https://phabricator.kde

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 I would have expect

[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 https://phabricator.kde.org/settings/panel/emailp

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

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

[Differential] [Request, 20 lines] D1501: [autotest] Work around "cannot assign undefined" warning

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 both tests cases in the qml file dialog test are run together, there is a warning: file:///opt/qt5

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

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 in KWayland [1] an

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

[Differential] [Request, 8 lines] D1487: Don't depend on operation mode in WindowPixmap

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 To check which code path we should take just check whether - we have a surface -> Wayland - we hav

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

[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&id=3536 BRANCH autotest-memory-manageme

[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 https:

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

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 and

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