Re: Review Request 118512: Turn KFileMetaData into a Framework.

2014-06-16 Thread Matthew Dawson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118512/ --- (Updated June 16, 2014, 5:44 p.m.) Status -- This change has been di

Re: Review Request 118678: [ki18n] Allow querying for translations of a specific domain

2014-06-16 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118678/ --- (Updated June 16, 2014, 9:32 p.m.) Status -- This change has been ma

Re: Review Request 118678: [ki18n] Allow querying for translations of a specific domain

2014-06-16 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118678/#review60213 --- This review has been submitted with commit d09bfbd3949688c3ea

Re: Re: kwindowsystem high level functions

2014-06-16 Thread Damian Ivanov
And in case this was a hint I should contribute to a wayland kwindowsystem backend, my c is awful, but I'm learning and believe I will when I can. :) ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 118753: Handle mouse wheel in PlasmaComponents.Slider

2014-06-16 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118753/#review60207 --- This review has been submitted with commit 76175fafa6d3a38d73

Re: Review Request 118753: Handle mouse wheel in PlasmaComponents.Slider

2014-06-16 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118753/ --- (Updated June 16, 2014, 6:07 p.m.) Status -- This change has been ma

Re: Review Request 118678: [ki18n] Allow querying for translations of a specific domain

2014-06-16 Thread Chusslove Illich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118678/#review60206 --- Ship it! Ship It! - Chusslove Illich On June 16, 2014, 4:4

Jenkins build is back to normal : plasma-workspace_master_qt5 #434

2014-06-16 Thread KDE CI System
See ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Re: kwindowsystem high level functions

2014-06-16 Thread Damian Ivanov
Yes, I absolutely will, I also have a wrapper that exposes kwindosystem and kwindowinfo (and few netwm ones) functions to qml https://github.com/damianatorrpm/qmltaskbar (used by https://github.com/damianatorrpm/qmlpanel - old revision there, uploading new one soon) It is a complete rewrite of my

Re: Re: kwindowsystem high level functions

2014-06-16 Thread Aleix Pol
On Mon, Jun 16, 2014 at 4:45 PM, Damian Ivanov wrote: > Ok. > >Whether closing a window is something which will be needed for Wayland in > >KWindowSystem is quite questionable. If at all it's the taskmanager, no > other > >client should be allowed to perform such a task. > I hope you consider for

Re: Re: kwindowsystem high level functions

2014-06-16 Thread Damian Ivanov
Ok. >Whether closing a window is something which will be needed for Wayland in >KWindowSystem is quite questionable. If at all it's the taskmanager, no other >client should be allowed to perform such a task. I hope you consider for the future non-plasma taskmanagers using kwindowsystem (my fingers

Re: Review Request 118678: [ki18n] Allow querying for translations of a specific domain

2014-06-16 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118678/ --- (Updated June 16, 2014, 2:43 p.m.) Review request for Plasma, Chusslove I

Re: Review Request 118387: Bump systemsettingsview SOVERSION to 5

2014-06-16 Thread Hrvoje Senjan
> On June 16, 2014, 8:54 a.m., Ben Cooksley wrote: > > As systemsettingsview is unlikely to break BC any further in the 4.x > > series, could the bump be to SOVERSION 3 instead please? > > Otherwise, this looks fine to go in from my perspective. off course, i've just used 5, as the rest of the

Re: Review Request 118763: Remove find_package(XCB) call as it is handled already by the top-level cmake file

2014-06-16 Thread Bernd Steinhauser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118763/ --- (Updated June 16, 2014, 2:07 p.m.) Review request for kde-workspace, Plas

Re: Review Request 117751: Improve dependencies.

2014-06-16 Thread Michael Palimaka
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117751/ --- (Updated June 16, 2014, 1:55 p.m.) Status -- This change has been ma

Re: Re: kwindowsystem high level functions

2014-06-16 Thread Martin Gräßlin
On Monday 16 June 2014 15:34:13 Damian Ivanov wrote: > I mean you read a property from one class and write to it in a differen > class? How can I use kwindowsystem to close a window in wayland (isn't > QX11Info::dispay required in netwm) ? not at all as there is no Wayland backend in KWindowSystem

Build failed in Jenkins: plasma-workspace_master_qt5 #433

2014-06-16 Thread KDE CI System
See Changes: [mgraesslin] Adjust KSMServer to renamed KWin binary -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER

Build failed in Jenkins: plasma-workspace_master_qt5 #432

2014-06-16 Thread KDE CI System
See Changes: [lukas] add missing and fix Messages.sh -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER LINBUILDER)

Re: Review Request 118574: Adjust componentchooser for renamed KWin binary

2014-06-16 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118574/ --- (Updated June 16, 2014, 1:39 p.m.) Status -- This change has been ma

Re: Review Request 118574: Adjust componentchooser for renamed KWin binary

2014-06-16 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118574/#review60198 --- This review has been submitted with commit ba346b6c2d17dff7f6

Re: Review Request 118482: Adjust ksmserver for renamed KWin binary

2014-06-16 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118482/ --- (Updated June 16, 2014, 1:39 p.m.) Status -- This change has been ma

Re: Review Request 118482: Adjust ksmserver for renamed KWin binary

2014-06-16 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118482/#review60197 --- This review has been submitted with commit 6cbbe0a2faddf75fdf

Build failed in Jenkins: plasma-workspace_master_qt5 #431

2014-06-16 Thread KDE CI System
See Changes: [lukas] add missing Messages.sh, fix catalog loading -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER

Re: kwindowsystem high level functions

2014-06-16 Thread Damian Ivanov
I mean you read a property from one class and write to it in a differen class? How can I use kwindowsystem to close a window in wayland (isn't QX11Info::dispay required in netwm) ? (don't get me wrong, I have great respect for your work and I do not remotely understand the complexities like you do

Build failed in Jenkins: plasma-workspace_master_qt5 #430

2014-06-16 Thread KDE CI System
See Changes: [aleixpol] Fix some problems in the lock screen -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER LINB

Re: Review Request 118512: Turn KFileMetaData into a Framework.

2014-06-16 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118512/#review60193 --- I just realized that there is another review about this who is

Re: kwindowsystem high level functions

2014-06-16 Thread Martin Gräßlin
On Monday 16 June 2014 15:14:08 Damian Ivanov wrote: > I think that Thomas Luebking agrred with me on this: > http://osdir.com/ml/kde-devel/2014-02/msg00063.html > > except closeWindow I also see missing > > void setBlockingCompositing(bool active); > void setShowingDesktop(bool showing); > v

kwindowsystem high level functions

2014-06-16 Thread Damian Ivanov
I think that Thomas Luebking agrred with me on this: http://osdir.com/ml/kde-devel/2014-02/msg00063.html except closeWindow I also see missing void setBlockingCompositing(bool active); void setShowingDesktop(bool showing); void setNumberOfDesktops(int numberOfDesktops); It would be only natu

Build failed in Jenkins: plasma-workspace_master_qt5 #429

2014-06-16 Thread KDE CI System
See Changes: [notmart] remove micro configure button -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER LINBUILDER)

Build failed in Jenkins: plasma-workspace_master_qt5 #428

2014-06-16 Thread KDE CI System
See Changes: [notmart] sensible default sizes [notmart] friendlier window title -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlav

Build failed in Jenkins: plasma-workspace_master_qt5 #427

2014-06-16 Thread KDE CI System
See Changes: [mgraesslin] [krunner] Add Qt::Key_Search as a global shortcut -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3

Re: Review Request 118779: [krunner] Add Qt::Key_Search as a global shortcut

2014-06-16 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118779/ --- (Updated June 16, 2014, 12:44 p.m.) Status -- This change has been m

Re: Review Request 118779: [krunner] Add Qt::Key_Search as a global shortcut

2014-06-16 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118779/#review60191 --- This review has been submitted with commit a68bb91fef6190fca8

Re: Review Request 118779: [krunner] Add Qt::Key_Search as a global shortcut

2014-06-16 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118779/#review60188 --- Ship it! Ship It! - Vishesh Handa On June 16, 2014, 11:45

Review Request 118779: [krunner] Add Qt::Key_Search as a global shortcut

2014-06-16 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118779/ --- Review request for Plasma and Vishesh Handa. Repository: plasma-workspace

Build failed in Jenkins: plasma-workspace_master_qt5 #426

2014-06-16 Thread KDE CI System
See Changes: [mgraesslin] Don't install ksyncdbusenv to /kf5 in libexecdir -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3

Re: Review Request 118730: [startkde] Introduce a ksyncdbusenv helper application

2014-06-16 Thread Martin Gräßlin
> On June 14, 2014, 9:12 p.m., Hrvoje Senjan wrote: > > startkde/ksyncdbusenv/CMakeLists.txt, line 5 > > > > > > it was my understanding that KF5_LIBEXEC_INSTALL_DIR is 'reserved' for > > KF5, and rest of dependin

Re: Minutes Monday Plasma hangout

2014-06-16 Thread Marco Martin
On Monday 16 June 2014, Sebastian Kügler wrote: > Marco: > - "mindless" bugfixing mode > - Monochrome kickoff tabbar > - Fixes in dialog (yes, more of those) - startup fixes: no random opening systray at startup - less animations, applets jumping around the panel at startup -- Marco Martin __

Minutes Monday Plasma hangout

2014-06-16 Thread Sebastian Kügler
Monday Hangout 16.6.2013 Present: Marco, Martin Gräßlin, Jens, Sebastian, Vishesh, Aleix Jens: - Had a meeting within the VDG, will send results around - Trying to beat Firefox into compliance with QtCurve - Lots of design work of systemsettings going on - Things are moving nicely overall Aleix:

Build failed in Jenkins: plasma-workspace_master_qt5 #425

2014-06-16 Thread KDE CI System
See Changes: [me] Move the Baloo search runner here -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 3 (PACKAGER LINBUILDER) i

Re: Review Request 118696: Remove unused dependencies.

2014-06-16 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118696/#review60181 --- This review has been submitted with commit fab88696dde52f1e0d

Re: Review Request 118696: Remove unused dependencies.

2014-06-16 Thread Michael Palimaka
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118696/ --- (Updated June 16, 2014, 10:14 a.m.) Status -- This change has been m

Re: Review Request 118678: [ki18n] Allow querying for translations of a specific domain

2014-06-16 Thread Chusslove Illich
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118678/#review60178 --- src/klocalizedstring.h

Re: Re: Re: Re: statusNotifier hide behaviour inconsistent

2014-06-16 Thread Damian Ivanov
Hmm ok. If I come up with a better idea I'll let you know :) ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Re: Re: Re: statusNotifier hide behaviour inconsistent

2014-06-16 Thread Martin Gräßlin
On Monday 16 June 2014 10:39:34 Damian Ivanov wrote: > It's ok if that behavior is for all wayland window manager's. > Every window manager supplies a list with known clients for the > copy/paste stack. > Maybe this is something that should be done as freedesktop standard > before it's too late...

Re: Re: Re: statusNotifier hide behaviour inconsistent

2014-06-16 Thread Damian Ivanov
It's ok if that behavior is for all wayland window manager's. Every window manager supplies a list with known clients for the copy/paste stack. Maybe this is something that should be done as freedesktop standard before it's too late... ___ Plasma-devel ma

Re: Re: Re: statusNotifier hide behaviour inconsistent

2014-06-16 Thread Martin Gräßlin
On Monday 16 June 2014 10:23:30 Damian Ivanov wrote: > > For Klipper this will happen for 5.1. Whether it's worth to keep klipper > > as a standalone application I'm not yet convinced, especially given that > > klipper will most likely be bound to KWin in a Wayland world. > > Just ot of curiosity

Re: Re: statusNotifier hide behaviour inconsistent

2014-06-16 Thread Damian Ivanov
> For Klipper this will happen for 5.1. Whether it's worth to keep klipper as a > standalone application I'm not yet convinced, especially given that klipper > will most likely be bound to KWin in a Wayland world. Just ot of curiosity what part of Klipper would require kwin? If it's for grabbing th

Re: Review Request 118693: [ksmserver/screenlocker] Use ScreenSaver key as shortcut

2014-06-16 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118693/ --- (Updated June 16, 2014, 7:19 a.m.) Status -- This change has been ma

Re: Review Request 118693: [ksmserver/screenlocker] Use ScreenSaver key as shortcut

2014-06-16 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118693/#review60166 --- This review has been submitted with commit 82f15720a8523031fa

Re: Review Request 118730: [startkde] Introduce a ksyncdbusenv helper application

2014-06-16 Thread Martin Gräßlin
> On June 14, 2014, 9:12 p.m., Hrvoje Senjan wrote: > > startkde/ksyncdbusenv/CMakeLists.txt, line 5 > > > > > > it was my understanding that KF5_LIBEXEC_INSTALL_DIR is 'reserved' for > > KF5, and rest of dependin

Re: Re: statusNotifier hide behaviour inconsistent

2014-06-16 Thread Martin Gräßlin
On Sunday 15 June 2014 13:36:52 Damian Ivanov wrote: > Thanks Martin for the answer. > > >The long term plan for the strange parts like klipper and kmix is to > >transform them to a Plasmoid. > I hope this won't happen any time soon, I use them in not plasma > (their StatusNotifiers) :) For Klipp