[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

[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=4864 REVISION DETAIL

[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

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,

[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

[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

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

[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

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

[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

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

[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

[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

[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

[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

[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

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

[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

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

[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

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:

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,

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

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

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

[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=4848 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:

[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=4847 BRANCH kwinqml REVISION DETAIL https://phabricator.kde.org/D1989 AFFECTED FILES

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

[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

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

[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

[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

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

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

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

[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=4843 REVISION

[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

[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=4842#toc REPOSITORY

[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

[Differential] [Closed] D2028: Paint the software cursor directly in SceneQPainter

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWIN758d41d6bfa5: Paint the software cursor directly in SceneQPainter (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2028?vs=4787=4840 REVISION

[Differential] [Closed] D2025: [platforms] Call setSoftWareCursor in init instead of ctor

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWINfeadcea6cea7: [platforms] Call setSoftWareCursor in init instead of ctor (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2025?vs=4783=4837

[Differential] [Closed] D2026: [platforms/drm] If mapping a DrmBuffer for a cursor fails, fallback to software cursor

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWIN7ea84cd34642: [platforms/drm] If mapping a DrmBuffer for a cursor fails, fallback to… (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE

[Differential] [Closed] D2027: Render cursor in multi-screen setup in QPainter Compositor

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWIN23fb02cce20a: Render cursor in multi-screen setup in QPainter Compositor (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2027?vs=4785=4839

[Differential] [Closed] D2024: Properly clip the damage region and windows in SceneQPainter

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWINfd19db0fbeae: Properly clip the damage region and windows in SceneQPainter (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2024?vs=4782=4836

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

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWAYLAND1837b8d4c391: Add a parent_window event to Plasma Window interface (authored by graesslin). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D1779?vs=4264=4834#toc REPOSITORY rKWAYLAND

[Differential] [Closed] D2030: Add a ToolTip role to PlasmaShellSurface

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWAYLANDa601cf3817e8: Add a ToolTip role to PlasmaShellSurface (authored by graesslin). REPOSITORY rKWAYLAND KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2030?vs=4795=4835

[Differential] [Closed] D2037: Properly handle destroying a Pointer resource

2016-06-29 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWAYLAND9651216beecd: Properly handle destroying a Pointer resource (authored by graesslin). REPOSITORY rKWAYLAND KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2037?vs=4810=4833

[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 #25 from Fabian Köster --- (In reply to Kai Uwe Broulik from comment #23) > I just updated the patch at [1] to clear idle timeouts when the session > becomes inactive. > > [1] https://phabricator.kde.org/D2033