[Differential] [Closed] D2010: On Wayland never try to create a WindowPixmap from an X11 pixmap

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWINf79c3f244d57: On Wayland never try to create a WindowPixmap from an X11 pixmap (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2010?vs=4739&id

[Differential] [Closed] D1973: Support for syncing the clipboard from X11 to Wayland and vice versa

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWIN3493e9765504: Support for syncing the clipboard from X11 to Wayland and vice versa (authored by graesslin). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D1973?vs=4737&id=4842#toc REPOSITORY

[Differential] [Closed] D2031: [kwindowsystem] Add support for NET::Tooltip

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWAYLANDINTEGRATIONc671900d9d24: [kwindowsystem] Add support for NET::Tooltip (authored by graesslin). REPOSITORY rKWAYLANDINTEGRATION Frameworks integration plugin using KWayland CHANGES SINCE LAST UPD

[Differential] [Closed] D2032: Add support for ToolTip Role on PlasmaSurface

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWINc9984094dec2: Add support for ToolTip Role on PlasmaSurface (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2032?vs=4798&id=4843 REVISION DET

[Differential] [Closed] D1780: Expose the transientFor relationship to PlasmaWindowInterface

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWINaa215e12ccdc: Expose the transientFor relationship to PlasmaWindowInterface (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1780?vs=4245&id=48

[Powerdevil] [Bug 364275] sleep and hibernate gone after update

2016-06-29 Thread via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364275 dominikdanel...@gmail.com changed: What|Removed |Added CC||dominikdanel...@gmail.com --- Commen

[Powerdevil] [Bug 364275] sleep and hibernate gone after update

2016-06-29 Thread Kai Uwe Broulik via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364275 Kai Uwe Broulik changed: What|Removed |Added CC||k...@privat.broulik.de Status|UNC

[Differential] [Closed] D2034: Set system tray applet to RequiresInput when a popup is showing

2016-06-29 Thread davidedmundson (David Edmundson)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACE7efc1c05d60d: Set system tray applet to RequiresInput when a popup is showing (authored by davidedmundson). REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE https:

[Differential] [Commented On] D1779: Add a parent_window event to Plasma Window interface

2016-06-29 Thread hein (Eike Hein)
hein added a comment. Just for the record: Martin's reply convinced me, so +1 after the fact. Sorry for this slipping through the cracks. REPOSITORY rKWAYLAND KWayland REVISION DETAIL https://phabricator.kde.org/D1779 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailp

Jenkins-kde-ci: plasma-workspace Plasma-5.7 stable-kf5-qt5 » Linux,gcc - Build # 16 - Still Failing!

2016-06-29 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-workspace%20Plasma-5.7%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/16/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 29 Jun 2016 09:23:58 + Build duration: 53 sec CHANGE SET Revision 7efc1c05d60de588a7504715b

[Differential] [Commented On] D1989: Introduce QQuickItem to nest kwin_wayland

2016-06-29 Thread bshah (Bhushan Shah)
bshah added a comment. Some minor issues I noticed.. INLINE COMMENTS > CMakeLists.txt:13 > + Qt5::Quick > + KF5::WaylandClient > + KF5::WaylandServer Not required, you just use WaylandServer API. > kwinplugin.h:1 > +#ifndef KWINPLUGIN_H > +#define KWINPLUGIN_H This and all other header

[Differential] [Requested Changes To] D1989: Introduce QQuickItem to nest kwin_wayland

2016-06-29 Thread bshah (Bhushan Shah)
bshah requested changes to this revision. bshah added a reviewer: bshah. This revision now requires changes to proceed. REPOSITORY rKWIN KWin REVISION DETAIL https://phabricator.kde.org/D1989 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: bdhruve, #plas

[Differential] [Updated, 225 lines] D1989: Introduce QQuickItem to nest kwin_wayland

2016-06-29 Thread bdhruve (Bhavisha Dhruve)
bdhruve updated this revision to Diff 4847. bdhruve added a comment. Fixed issues mentioned by @bshah REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1989?vs=4797&id=4847 BRANCH kwinqml REVISION DETAIL https://phabricator.kde.org/D1989 AFFECTED FILES

Review Request 128316: Make use of QQuickItem::setSize instead of wight and height indpedently.

2016-06-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128316/ --- Review request for KDE Frameworks and Plasma. Repository: plasma-framewor

[Differential] [Updated, 225 lines] D1989: Introduce QQuickItem to nest kwin_wayland

2016-06-29 Thread bdhruve (Bhavisha Dhruve)
bdhruve updated this revision to Diff 4848. bdhruve added a comment. Fixed copyright header. REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1989?vs=4847&id=4848 BRANCH kwinqml REVISION DETAIL https://phabricator.kde.org/D1989 AFFECTED FILES config-kw

Re: Review Request 128316: Make use of QQuickItem::setSize instead of width and height indpedently.

2016-06-29 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128316/#review96951 --- Ship it! I think you found an undocumented feature. The do

Re: Review Request 128316: Make use of QQuickItem::setSize instead of width and height indpedently.

2016-06-29 Thread Kai Uwe Broulik
> On Juni 29, 2016, 11:05 vorm., Mark Gaiser wrote: > > I think you found an undocumented feature. > > The docs don't mention it, not even in the list of all members + inherited > > ones: http://doc.qt.io/qt-5/qquickitem-members.html > > > > But if i add a QQuickItem in my project and look up t

Jenkins-kde-ci: plasma-desktop Plasma-5.7 stable-kf5-qt5 » Linux,gcc - Build # 16 - Still Failing!

2016-06-29 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.7%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/16/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 29 Jun 2016 11:01:18 + Build duration: 7 min 15 sec CHANGE SET Revision df230872204f1457192bb

Re: Review Request 128316: Make use of QQuickItem::setSize instead of width and height indpedently.

2016-06-29 Thread Mark Gaiser
> On jun 29, 2016, 11:05 a.m., Mark Gaiser wrote: > > I think you found an undocumented feature. > > The docs don't mention it, not even in the list of all members + inherited > > ones: http://doc.qt.io/qt-5/qquickitem-members.html > > > > But if i add a QQuickItem in my project and look up the

Re: Review Request 128316: Make use of QQuickItem::setSize instead of width and height indpedently.

2016-06-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128316/#review96954 --- Ship it! Ship It! - Marco Martin On June 29, 2016, 10:4

[Powerdevil] [Bug 354250] "Switch User" sometimes sends computer to Standby

2016-06-29 Thread Oliver Henshaw via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354250 --- Comment #26 from Oliver Henshaw --- (In reply to Martin Gräßlin from comment #24) > From comment @18 it looks like X freezes while being on the inactive > session. As Fedora is mentioned I assume it's an X not run as root? If X > freezes all xcb req

[Powerdevil] [Bug 354250] "Switch User" sometimes sends computer to Standby

2016-06-29 Thread Martin Gräßlin via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354250 --- Comment #27 from Martin Gräßlin --- both threads are waiting (X can do multi-threading, that's like wow I'm impressed). Though the one is a graphics thread and is somewhere in radeon. That could be an indication for being blocked and waiting for som

Re: Long term support release for Plasma?

2016-06-29 Thread Gastón Cocco
Good idea. El mié., 29 de jun. de 2016 11:29 AM, Martin Gräßlin escribió: > Am 2016-06-29 15:34, schrieb Philip Muskovac: > >> * how long should a LTS be supported? > > I would be happy with your proposal, but it would be nice if security > > issues could be evaluated for the previous LTS as wel

[Powerdevil] [Bug 354250] "Switch User" sometimes sends computer to Standby

2016-06-29 Thread Oliver Henshaw via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354250 --- Comment #28 from Oliver Henshaw --- Note that this wasn't a problem on F21, so this is a regression somewhere, not necessarily a misdesign feature. Could be an X regression, could be a kde 4 -> 5 regression, could be a qt 4 -> 5 regression. Note th

Re: Long term support release for Plasma?

2016-06-29 Thread Martin Gräßlin
Am 2016-06-29 15:34, schrieb Philip Muskovac: * how long should a LTS be supported? I would be happy with your proposal, but it would be nice if security issues could be evaluated for the previous LTS as well. Doesn't need a release, a note in the advisory would be sufficient. Just as a note:

[Powerdevil] [Bug 354250] "Switch User" sometimes sends computer to Standby

2016-06-29 Thread Fabian Köster via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354250 --- Comment #29 from Fabian Köster --- Please note that I do not use Fedora but Gentoo Linux which still executes X as root. "ps aux | grep /usr/bin/X" confirms that: root 2229 2.2 0.5 407300 89944 tty1 Ssl+ 15:39 1:32 /usr/bin/X -nolist

[Differential] [Request, 186 lines] D2044: WIP: [KSMServer] Load logout dialog after startup rather than on-demand

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, sitter. broulik set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This ensures the l

[Differential] [Commented On] D2044: WIP: [KSMServer] Load logout dialog after startup rather than on-demand

2016-06-29 Thread Martin Gräßlin
graesslin added a comment. I'm not sure whether that's the correct approach to the problem. It's a workaround and we have probably a bazillion other places where it breaks as well. IMHO the updates should be installed offline and AFAIU packagekit supports that. REPOSITORY rPLASMAWORKSPACE

[Differential] [Commented On] D2044: WIP: [KSMServer] Load logout dialog after startup rather than on-demand

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik added a comment. Harald, weren't you even looking into offline updates? REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2044 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, sitter Cc

[Differential] [Request, 4 lines] D2045: [Task Manager] Fix collapsing tooltip preview area for launchers

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, hein, bshah. broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY BUG: 364900 FIXE

[Differential] [Request, 147 lines] D2046: [autotests] Add a new test case which can verify the rendering of QPainter Scene

2016-06-29 Thread Martin Gräßlin
graesslin created this revision. graesslin added reviewers: KWin, Plasma on Wayland. Restricted Application added subscribers: kwin, plasma-devel. Restricted Application added projects: Plasma on Wayland, KWin. REVISION SUMMARY The idea behind this autotest is inspired by bug 356328 which produc

[Differential] [Request, 3 lines] D2047: [KDE Platform Theme] Set application layoutDirection based on locale

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, Plasma: Design. broulik set the repository for this revision to rPLASMAINTEGRATION Integration for Qt applications in Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. RE

[Differential] [Request, 7 lines] D2049: [Desktop Package] Adjust layout for right-to-left languages

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This ensures content of applet

[Differential] [Request, 12 lines] D2048: [System Tray] Adjust layout for right-to-left languages

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This ensures stuff in the

Review Request 128327: [Calendar] Flip arrow buttons on right-to-left languages

2016-06-29 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128327/ --- Review request for Plasma. Repository: plasma-framework Description ---

[Differential] [Requested Changes To] D2047: [KDE Platform Theme] Set application layoutDirection based on locale

2016-06-29 Thread davidedmundson (David Edmundson)
davidedmundson requested changes to this revision. davidedmundson added a reviewer: davidedmundson. davidedmundson added a comment. This revision now requires changes to proceed. Looking at QGuiApplication.cpp this is all manner of wrong. You are right we have a bug, but this really isn't

[Differential] [Commented On] D2047: [KDE Platform Theme] Set application layoutDirection based on locale

2016-06-29 Thread davidedmundson (David Edmundson)
davidedmundson added a comment. We don't have a bug. Make sure you have qttranslations build and installed. If you're using distro packages and they're not installing that, please tell them. REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL http

[Differential] [Commented On] D2047: [KDE Platform Theme] Set application layoutDirection based on locale

2016-06-29 Thread davidedmundson (David Edmundson)
davidedmundson added a comment. *built and installed REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D2047 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma:_design, #

[Differential] [Changed Subscribers] D2047: [KDE Platform Theme] Set application layoutDirection based on locale

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik added a subscriber: sitter. broulik added a comment. @sitter I have installed qttranslations5-l10n but it's version 5.5.1 so probably not from neon packages? Or is this because I'm on neon dev ed? REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETA

[Differential] [Accepted] D2048: [System Tray] Adjust layout for right-to-left languages

2016-06-29 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. This revision is now accepted and ready to land. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2048 EMAIL PREFERENCES https://phabricator.kde.org/settings/pan

Re: Review Request 128327: [Calendar] Flip arrow buttons on right-to-left languages

2016-06-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128327/#review96967 --- Ship it! Ship It! - Marco Martin On June 29, 2016, 8:34

[Differential] [Request, 4 lines] D2051: [Panel Containment] Fix containment size for right-to-left-layout

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY The X offset was already adjus

[Differential] [Updated, 2 lines] D2051: [Panel Containment] Fix containment size for right-to-left-layout

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik updated the summary for this revision. broulik updated the test plan for this revision. broulik updated this revision to Diff 4864. REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2051?vs=4863&id=4864 REVISION DETAIL https://phabricat

[Differential] [Abandoned] D2047: [KDE Platform Theme] Set application layoutDirection based on locale

2016-06-29 Thread broulik (Kai Uwe Broulik)
broulik abandoned this revision. broulik added a comment. User also reported that installing this package fixed the issue. It's Arch so probably expected that you need to install everything manually :P but on my Neon setup it should also work out of the box. REPOSITORY rPLASMAINTEGRATION I

[Differential] [Accepted] D2051: [Panel Containment] Fix containment size for right-to-left-layout

2016-06-29 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. This revision is now accepted and ready to land. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2051 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/e

[Differential] [Request, 277 lines] D2052: [autotests/integration] Simplify the cmake for creating a test

2016-06-29 Thread Martin Gräßlin
graesslin created this revision. graesslin added reviewers: KWin, Plasma on Wayland. Restricted Application added subscribers: kwin, plasma-devel. Restricted Application added projects: Plasma on Wayland, KWin. REVISION SUMMARY A dedicated cmake function is added to create a test case. It takes