[Breeze] [Bug 357649] Light text on light background in device notifier

2016-01-07 Thread Tom Warnke via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=357649 Tom Warnke changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Powerdevil] [Bug 357650] Power Management Window won't go away

2016-01-07 Thread Jim Hessin via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=357650 --- Comment #2 from Jim Hessin --- I am using Kubuntu 14.04 (LTS) - and whatever comes with that I'm not sure how to tell. On Thu, Jan 7, 2016 at 7:01 AM, Sebastian Kügler via KDE Bugzilla < bugzilla_nore...@kde.org> wrote: > https://bugs.kde.org/show

Re: Review Request 126669: [Task Manager] Don't show on which virtual desktop a window is if there is just one

2016-01-07 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126669/#review90786 --- Ship it! Ship It! - Eike Hein On Jan. 7, 2016, 10 p.m., Ka

Re: Review Request 126672: Fix most of Clazy warnings in plasma-framework

2016-01-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126672/#review90785 --- Looks good. +1 - Aleix Pol Gonzalez On Jan. 8, 2016, 2:34

Re: Review Request 126672: Fix most of Clazy warnings in plasma-framework

2016-01-07 Thread Sergey Popov
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126672/ --- (Updated Янв. 8, 2016, 1:20 д.п.) Review request for Plasma and Aleix Pol

Review Request 126672: Fix most of Clazy warnings in plasma-framework

2016-01-07 Thread Sergey Popov
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126672/ --- Review request for Plasma and Aleix Pol Gonzalez. Repository: plasma-fram

Review Request 126673: [Device Notifer] Rewrite delegates to use Layouts

2016-01-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126673/ --- Review request for Plasma and KDE Usability. Repository: plasma-workspace

Re: Review Request 126669: [Task Manager] Don't show on which virtual desktop a window is if there is just one

2016-01-07 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126669/#review90780 --- Thank you. +1 (similar thing could be done with activities, I

Re: Review Request 126300: Plasma Search KCM: display the runner descriptions

2016-01-07 Thread Heiko Tietze
> On Jan. 7, 2016, 10:42 a.m., Heiko Tietze wrote: > > Looks good from the usability POV. But what happens with very long text, > > like Finnish translation? In the HIG we wrote "Do not add line breaks. If > > necessary place an additional label below the check box." (which is worth > > to rev

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

2016-01-07 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-workspace%20Plasma-5.5%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/38/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 07 Jan 2016 22:10:27 + Build duration: 1 min 59 sec CHANGE SET Revision abb010cae5c6c0093ae

Review Request 126669: [Task Manager] Don't show on which virtual desktop a window is if there is just one

2016-01-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126669/ --- Review request for Plasma, KDE Usability and Eike Hein. Repository: plasm

Re: Review Request 126621: [Task Manager] Add support for Unity Launcher API and Application Jobs

2016-01-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126621/ --- (Updated Jan. 7, 2016, 9:21 nachm.) Review request for Plasma, KDE Usabil

Re: Review Request 126647: [Task Manager] Provide media controls in tooltips

2016-01-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126647/ --- (Updated Jan. 7, 2016, 9:05 nachm.) Review request for Plasma, KDE Usabil

Re: Review Request 126668: [notifications] Place the popup directly when it is displayed

2016-01-07 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126668/ --- (Updated Jan. 7, 2016, 9:08 p.m.) Review request for Plasma. Repository

Review Request 126668: [notifications] Place the popup directly when it is displayed

2016-01-07 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126668/ --- Review request for Plasma. Repository: plasma-workspace Description ---

[Powerdevil] [Bug 357449] laptop screen is always dimmed after 5 min

2016-01-07 Thread via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=357449 g...@section9.follonica.org changed: What|Removed |Added CC||g...@section9.follonica.org --- Co

Re: KDE Discover binary & library names

2016-01-07 Thread Jonathan Riddell
On Thu, Jan 07, 2016 at 05:59:50PM +0100, Matthias Klumpp wrote: > So, possible solutions would be, for the binary: > 1) Keep the name "muon-discover", although there is no connection to > Muon anymore > 2) Use "plasma-discover" > 3) Use "kde-discover" > 4) Use "kdiscover" > 5) ${ANY_OTHER_NAM

Re: Review Request 126652: [Units] Return at least 1ms for durations

2016-01-07 Thread David Edmundson
> On Jan. 7, 2016, 4:32 p.m., Kai Uwe Broulik wrote: > > I guess we should rather get https://codereview.qt-project.org/#/c/145237/ > > in :) nah, that'll be Qt 5.6 at the earliest. +1 from me on the patch. It's fine to have workarounds if we know they'll only be temporary workarounds. - Dav

Re: Review Request 126591: [Task Manager] Adjust ToolTip when no thumbnail is available

2016-01-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126591/ --- (Updated Jan. 7, 2016, 4:32 nachm.) Status -- This change has been d

Re: Review Request 126660: Avoid finding the same package multiple times from different paths.

2016-01-07 Thread Andreas Hartmetz
> On Jan. 7, 2016, 3:31 p.m., Sebastian Kügler wrote: > > src/kpackage/packageloader.cpp, line 190 > > > > > > What does the category have to do with this? We should only be going by > > the id (the plugin name)

Re: Review Request 126660: Avoid finding the same package multiple times from different paths.

2016-01-07 Thread Andreas Hartmetz
> On Jan. 7, 2016, 3:31 p.m., Sebastian Kügler wrote: > > src/kpackage/packageloader.cpp, line 187 > > > > > > This doesn't actually add anything. A better name would IMO be: > > alreadyListed() or something alo

Re: Review Request 126660: Avoid finding the same package multiple times from different paths.

2016-01-07 Thread Andreas Hartmetz
> On Jan. 7, 2016, 3:31 p.m., Sebastian Kügler wrote: > > Wouldn't it be way easier to simply check if the plugin is already in lst? > > > > Your approach looks very complex for just that... O(n) vs. O(n^2) - Andreas --- This is an aut

Re: Review Request 126660: Avoid finding the same package multiple times from different paths.

2016-01-07 Thread Andreas Hartmetz
> On Jan. 7, 2016, 3:31 p.m., Sebastian Kügler wrote: > > src/kpackage/packageloader.cpp, line 187 > > > > > > This doesn't actually add anything. A better name would IMO be: > > alreadyListed() or something alo

Re: Review Request 126660: Avoid finding the same package multiple times from different paths.

2016-01-07 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126660/#review90755 --- Wouldn't it be way easier to simply check if the plugin is alr

Re: AW: some feedback from a mobile application development team

2016-01-07 Thread Sebastian Kügler
On Wednesday, January 06, 2016 08:11:47 PM Kai Uwe Broulik wrote: > > ‎qinputmethod does have keyboard geometry even in 5.5, on linux it works > > fine... so that doesn't work on android? > > ‎Yes, Google in their infinite wisdom believe that an app should never ever > need to probe for the keyb

Re: some feedback from a mobile application development team

2016-01-07 Thread Sebastian Kügler
On Wednesday, January 06, 2016 10:48:10 AM Dirk Hohndel wrote: > On Wed, Jan 06, 2016 at 04:39:33PM +0100, Marco Martin wrote: > > On Wednesday 06 January 2016, Dirk Hohndel wrote: > > > Building for Android is painful (as Sebastian will confirm). Adding > > > something where an application after b

Re: modifying the drawer menus

2016-01-07 Thread Kevin Krammer
On Wednesday, 2016-01-06, 23:33:32, Marco Martin wrote: > On Wednesday 06 January 2016, Kai Uwe Broulik wrote: > > Don't we handle the visible and enabled property of Action? That should > > work for enabling / disabling certain drawer entries if in edit mode.‎ > > Irregardless of that usecase I wo

Re: Review Request 126654: rework ColorScope, hopefully fix black on black battery on phone

2016-01-07 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126654/ --- (Updated Jan. 7, 2016, 10:07 a.m.) Status -- This change has been ma

Re: [PATCH] Mobile components: use back button to close global and context drawers

2016-01-07 Thread Marco Martin
On Thursday 07 January 2016, Dirk Hohndel wrote: > This is a possible patch for the first issue I just reported :-) > thanks, I appled it :) -- Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plas

Re: Review Request 126300: Plasma Search KCM: display the runner descriptions

2016-01-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126300/#review90738 --- +1 for the idea A bunch of coding nitpicks below. kcms/runn

Re: Review Request 126621: [Task Manager] Add support for Unity Launcher API and Application Jobs

2016-01-07 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126621/#review90739 --- Note to myself: the type for "count", for some reason, is 64 b