Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Martin Gräßlin
> On July 1, 2014, 10:15 p.m., David Edmundson wrote: > > You know my feelings on this. > > I will rehash them here if you really want it public. > > > > >It's our task to provide the *best possible* user experience > > (emphasis mine) > > > > before this patch: > > http://imgur.com/RjtXN0n,

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Martin Gräßlin
> On July 1, 2014, 8:04 p.m., Hugo Pereira Da Costa wrote: > > @Martin > > yes we can (tm). But then I'll get complains (got some already in the past) > > from people using oxygen-gtk on gnome (or other DEs). > > so there needs to be a way to communicate. (or as thomas suggested, some > > syst

[KDE Bugtracking System] REMINDER: current Plasma regressions

2014-07-01 Thread bugzilla_noreply
Please find below a list of the current regressions reported for Plasma. This is a weekly reminder. This search was scheduled by myr...@kde.org. Plasma regressions -- Bug 301424: https://bugs.kde.org/show_bug.cgi?id=301424 Priority: NOR Severity: normal Platform: openSUS

Review Request 119081: Install all kioslaves to $libdir/plugins/kf5/kio

2014-07-01 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119081/ --- Review request for Plasma. Repository: kio-extras Description ---

[plasma-workspace] /: Move path argument into KIconLoader ctor

2014-07-01 Thread Sebastian Kügler
Git commit 95ce6207c596714e3e9e8578dcc80fd8832b934d by Sebastian Kügler. Committed on 01/07/2014 at 22:24. Pushed by sebas into branch 'master'. Move path argument into KIconLoader ctor This is needed to build against KIconThemes 6a75883159bbeb9, which reverts the commit that adds the path to the

Re: Kioslave repos

2014-07-01 Thread Jonathan Riddell
On Tue, Jul 01, 2014 at 11:25:11PM +0200, David Faure wrote: > On Sunday 27 April 2014 19:35:37 I wrote: > > > * place that repo in kde/ or kde/ in the projects.kde.org > > > hierarchy, so that it gets released with the KDE Applications, NOT with > > > the > > > workspace product. Support for kio_f

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Thomas Lübking
> On July 1, 2014, 8:15 p.m., David Edmundson wrote: > > You know my feelings on this. > > I will rehash them here if you really want it public. > > > > >It's our task to provide the *best possible* user experience > > (emphasis mine) > > > > before this patch: > > http://imgur.com/RjtXN0n,V

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119062/#review61427 --- You know my feelings on this. I will rehash them here if you

Jenkins build is back to normal : plasma-desktop_master_qt5 #401

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

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Thomas Lübking
> On July 1, 2014, 6:04 p.m., Hugo Pereira Da Costa wrote: > > @Martin > > yes we can (tm). But then I'll get complains (got some already in the past) > > from people using oxygen-gtk on gnome (or other DEs). > > so there needs to be a way to communicate. (or as thomas suggested, some > > syst

Build failed in Jenkins: plasma-desktop_master_qt5 #400

2014-07-01 Thread KDE CI System
See Changes: [notmart] use only addAppDir -- [...truncated 335 lines...] Call Stack (most recent call first): CMakeLists.txt:28 (find_package) -- Could NOT find KF5NewStuff (found versio

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Hugo Pereira Da Costa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119062/#review61412 --- @Martin yes we can (tm). But then I'll get complains (got some

Re: Moving parts of kde-baseapps to plasma-workspace

2014-07-01 Thread Eike Hein
On 07/01/2014 06:49 PM, Eike Hein wrote: Yeah, this can die. Folder now lives in plasma-desktop. -> Deleted from kde-baseapps.git. Cheers, Eike ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Moving parts of kde-baseapps to plasma-workspace

2014-07-01 Thread Vishesh Handa
On Tue, Jul 1, 2014 at 6:49 PM, Eike Hein wrote: > Yeah, this can die. Folder now lives in plasma-desktop. kde-baseapps/plasma has been removed http://commits.kde.org/kde-baseapps/df78336e6974996dffbc13c77a0d28061ba283d2 -- Vishesh Handa ___ Plasm

Re: Moving parts of kde-baseapps to plasma-workspace

2014-07-01 Thread Eike Hein
On 07/01/2014 06:36 PM, Vishesh Handa wrote: 2. Folder View Applet - This seems to be covered with plasma-desktop/containments/folder. So, should we just delete kde-baseapps/plasma? @Eike: I think you worked on this, right? Yeah, this can die. Folder now lives in plasma-desktop. We additio

Re: new systemsettings categorization landed

2014-07-01 Thread Vishesh Handa
On Tue, Jul 1, 2014 at 6:25 PM, Ivan Čukić wrote: > > I don't understand why "File Search" (formerly Desktop Search) should go > > under Applications. > > The Applications category is problematic, but if that came out from the > user > survey, who are we to complain. We have no way of knowing h

Moving parts of kde-baseapps to plasma-workspace

2014-07-01 Thread Vishesh Handa
Hey guys We need to ship parts of kde-baseapps before the release in 2 weeks. The following components are required in a Plasma Workspace - 1. kdepasswd - Contains the KCM for changing the real name and picture. This name is used by kickoff and is prominently displayed. It also contains an app c

Re: new systemsettings categorization landed

2014-07-01 Thread Ivan Čukić
Hi all, The new organization is surprisingly good. Surprisingly, because at for the first 5 seconds, I wanted to say 'awful' but was unable. There are a few things that could be fixed (not really organization, rather naming): - Do we really need the word 'Human' for the 'Input devices'? - We h

Re: new systemsettings categorization landed

2014-07-01 Thread Vishesh Handa
On Tue, Jul 1, 2014 at 5:36 PM, Heiko Tietze wrote: > Hi Vishesh & all, > > I agree that it makes more sense to place it with the desktop stuff. Maybe > we had application = file in mind? The only argument against desktop > behavior is that this topic gets somewhat bulky. And I'm not sure that >

Re: new systemsettings categorization landed

2014-07-01 Thread Thomas Pfeiffer
On 01.07.2014 17:42, Andres Silva wrote: Heiko Tietze wrote: Hi Vishesh & all, I agree that it makes more sense to place it with the desktop stuff. Maybe we had application = file in mind? The only argument against desktop behavior is that this topic gets somewhat bulky. And I'm not sure that u

Re: Review Request 118978: [screenlocker/greeter] Disable session management in the greeter

2014-07-01 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118978/#review61404 --- This review has been submitted with commit c9b0480069e650626a

Re: Review Request 118978: [screenlocker/greeter] Disable session management in the greeter

2014-07-01 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118978/ --- (Updated July 1, 2014, 3:53 p.m.) Status -- This change has been mar

Re: Review Request 118977: KSMServer: Do not autostart files

2014-07-01 Thread Vishesh Handa
> On June 27, 2014, 3:10 p.m., Rohan Garg wrote: > > I applied it locally here and krunner still starts up twice. > > Rohan Garg wrote: > So, krunner starting twice is another bug in session management, but > other apps like konsole only startup once after this patch > > Marco Martin wrote

Re: Review Request 118977: KSMServer: Do not autostart files

2014-07-01 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118977/ --- (Updated July 1, 2014, 3:24 p.m.) Review request for Plasma and Àlex Fies

Re: new systemsettings categorization landed

2014-07-01 Thread Vishesh Handa
Hey guys. I don't understand why "File Search" (formerly Desktop Search) should go under Applications. Current application which can be used to search for files - Dolphin. Current workspace stuff which can be used to search - KRunner, Milou, Kickoff and Kicker. It seems rather odd for it to be u

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119062/ --- (Updated July 1, 2014, 4:53 p.m.) Review request for kwin, Plasma and Hug

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Martin Gräßlin
> On July 1, 2014, 4:48 p.m., Thomas Lübking wrote: > > To reduce the visual issue (shadows) we could seek to inject > > > > ~/.config/gtk-3.0/gtk.css > > .window-frame {box-shadow: none; margin: 0px; padding:0px} > > /*GtkWindow { -GtkWindow-decoration-button-layout: "close:"; }*/ > > > > ie

Re: Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119062/#review61397 --- To reduce the visual issue (shadows) we could seek to inject

Jenkins build is back to normal : plasma-desktop_master_qt5 #397

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

Build failed in Jenkins: plasma-desktop_master_qt5 #396

2014-07-01 Thread KDE CI System
See Changes: [jr] remove knetattach icon, overlaps with kde-runtime from kdelibs4 land and its in oxygen icons anyway so unnecessary -- Started by remote host 2a01:4f8:160:9363::9 with note:

Review Request 119062: Add a script to enforce window decorations for GTK windows

2014-07-01 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119062/ --- Review request for kwin and Plasma. Repository: kwin Description --

Re: Review Request 118978: [screenlocker/greeter] Disable session management in the greeter

2014-07-01 Thread Martin Gräßlin
> On June 27, 2014, 11:16 p.m., David Edmundson wrote: > > ksmserver/screenlocker/greeter/main.cpp, line 65 > > > > > > why do we need it here too? > > Martin Gräßlin wrote: > Honestly: I do not know. But I o

new systemsettings categorization landed

2014-07-01 Thread Sebastian Kügler
Hey, So as the subject suggests, I've just landed the new categorization for systemsettings. Changes are all over the place, so if you just update single repositories, your systemsettings will be quite empty. If you're re-running kdesrc-build, you'll find a mess with old and new categorized int

Re: 5.0 and 5.1 Release Plans

2014-07-01 Thread Sebastian Kügler
On Tuesday, July 01, 2014 12:40:06 Marco Martin wrote: > On Tuesday 01 July 2014, Sebastian Kügler wrote: > > > > > > Why not the latest one? I don't see an advantage in restricting ourselves > > to older releases. We *want* people to use the latest and greatest, and I > > think we should mandate

Re: Review Request 118978: [screenlocker/greeter] Disable session management in the greeter

2014-07-01 Thread David Edmundson
> On June 27, 2014, 9:16 p.m., David Edmundson wrote: > > ksmserver/screenlocker/greeter/main.cpp, line 65 > > > > > > why do we need it here too? > > Martin Gräßlin wrote: > Honestly: I do not know. But I on

Re: Review Request 118978: [screenlocker/greeter] Disable session management in the greeter

2014-07-01 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118978/#review61367 --- Ship it! Ship It! - David Edmundson On June 27, 2014, 3:26

Re: 5.0 and 5.1 Release Plans

2014-07-01 Thread Marco Martin
On Tuesday 01 July 2014, Sebastian Kügler wrote: > > Why not the latest one? I don't see an advantage in restricting ourselves > to older releases. We *want* people to use the latest and greatest, and I > think we should mandate an up-to-date frameworks, when people are > installing Plasma. yeah

Re: Review Request 118974: UserAccount KCM: Change the name using the AccountManager

2014-07-01 Thread Vishesh Handa
> On July 1, 2014, 10:08 a.m., Commit Hook wrote: > > This review has been submitted with commit > > 77275f50241eb2f4e38051152074c379fa96fe84 by Vishesh Handa to branch > > frameworks. Alex gave me a ship it via IM. - Vishesh --- This

Re: Review Request 118974: UserAccount KCM: Change the name using the AccountManager

2014-07-01 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118974/ --- (Updated July 1, 2014, 10:08 a.m.) Status -- This change has been ma

Re: Review Request 118974: UserAccount KCM: Change the name using the AccountManager

2014-07-01 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118974/#review61357 --- This review has been submitted with commit 77275f50241eb2f4e3

Re: Review Request 118974: UserAccount KCM: Change the name using the AccountManager

2014-07-01 Thread Sebastian Kügler
> On July 1, 2014, 9:08 a.m., Sebastian Kügler wrote: > > kdepasswd/kcm/main.cpp, line 203 > > > > > > Perhaps convert to using KMessageWidget, while at it? > > Vishesh Handa wrote: > I rather not, that woul

Re: Review Request 118974: UserAccount KCM: Change the name using the AccountManager

2014-07-01 Thread Vishesh Handa
> On July 1, 2014, 9:08 a.m., Sebastian Kügler wrote: > > kdepasswd/kcm/main.cpp, line 203 > > > > > > Perhaps convert to using KMessageWidget, while at it? I rather not, that would make the patch much larger si

Re: 5.0 and 5.1 Release Plans

2014-07-01 Thread Sebastian Kügler
On Monday, June 30, 2014 19:55:54 Marco Martin wrote: > On Monday 30 June 2014, Aleix Pol wrote: > > > > It's quite unfortunate the fact that we'll have a more schedule for the > > frameworks and then the workspaces won't be able to take advantage of > > those. > > > > I don't think it makes sense

Re: Minutes Monday Plasma hangout

2014-07-01 Thread Sebastian Kügler
On Monday, June 30, 2014 13:04:20 Marco Martin wrote: > Minutes of hangout from 30th June 2014 > > Present: Ivan, Jens, David, Jonathan, Marco, Martin G, Vishesh [...] Sebastian: * Systemsettings new categorization, will push today * General bugfixing and testing * Some release promo preps --

Re: Plasmate Debian packaging issues

2014-07-01 Thread Sebastian Kügler
On Saturday, June 28, 2014 13:33:27 David Suárez wrote: > El Sábado, 28 de junio de 2014 13:30:34 Martin Graesslin escribió: > > sounds like a no-brainer to me. I know that we also have modeltest in KWin > > with an appropriate license. > > > > > > > > Do you have push privileges? If yes just go a

Re: Fwd: default wallpapers

2014-07-01 Thread Marco Martin
On Monday 30 June 2014, Marco Martin wrote: > besides the breeze wallpaper, there will be 3 extra wallpapers shipped by > default, that are the ones that won the contest. > the real big question, is: *where to put them?* that's one I don't have an > answer at the moment. > seems that ideally would

Re: Review Request 118974: UserAccount KCM: Change the name using the AccountManager

2014-07-01 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118974/#review61084 --- I'm not familiar with AccountsService, would appreciate someon

Re: Review Request 119053: KGlobalAccel fixes for loading shortcuts

2014-07-01 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119053/#review61346 --- This review has been submitted with commit 4caa63d655d603268d

Re: Review Request 119053: KGlobalAccel fixes for loading shortcuts

2014-07-01 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119053/ --- (Updated July 1, 2014, 8:37 a.m.) Status -- This change has been mar

Re: Review Request 119053: KGlobalAccel fixes for loading shortcuts

2014-07-01 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119053/#review61344 --- Ship it! Push it, after looking at my small optimization sugg

Review Request 119053: KGlobalAccel fixes for loading shortcuts

2014-07-01 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119053/ --- Review request for KDE Frameworks, Plasma and David Faure. Repository: kg