[Differential] [Accepted] D2656: [Kickoff] Lazy load more

2016-09-04 Thread hein (Eike Hein)
hein accepted this revision. hein added a comment. This revision is now accepted and ready to land. Sorry, brain fart :) REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2656 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferen

Re: Is it possible to know the panel's anchoring?

2016-09-04 Thread Michail Vourlakos
I see... this is interesting thanks Martin... no, the panel alignment 9and position in pixels) are properties of the view rather than the containment, so is not accepssible from there (from a c++ component it could try to blindly read properties from its view, but eew) -- Marco Martin

Re: Copy Activity - Create Activity by template?

2016-09-04 Thread Michail Vourlakos
thanks a lot Martin ! he could add a default setup javascrpit file in the look and feelpackage he's using. a proper named javascript file is executed every time a plasmoid is added, this is designed for distributions that want to have a different default or the plasmoid. (in look and feel pack

[Differential] [Commented On] D2656: [Kickoff] Lazy load more

2016-09-04 Thread broulik (Kai Uwe Broulik)
broulik added a comment. Dashboard? This is Kickoff, not Kicker REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2656 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, hein Cc: plasma-devel, leslie

[Differential] [Updated] D2656: [Kickoff] Lazy load more

2016-09-04 Thread hein (Eike Hein)
hein added a comment. Did you test Dashboard? REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2656 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, hein Cc: plasma-devel, lesliezhai, ali-mohamed,

[Differential] [Updated] D2658: Fix build failure introduced by the previous commit

2016-09-04 Thread Jan Kundrát
jkt added a dependent revision: D2660: Prefer an algorithm over an explicit for loop. REPOSITORY rPOWERDEVIL Powerdevil REVISION DETAIL https://phabricator.kde.org/D2658 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: jkt, broulik Cc: plasma-devel, lesl

[Differential] [Updated] D2660: Prefer an algorithm over an explicit for loop

2016-09-04 Thread Jan Kundrát
jkt added a dependency: D2658: Fix build failure introduced by the previous commit. REPOSITORY rPOWERDEVIL Powerdevil REVISION DETAIL https://phabricator.kde.org/D2660 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: jkt, broulik Cc: plasma-devel, leslie

[Differential] [Request, 9 lines] D2660: Prefer an algorithm over an explicit for loop

2016-09-04 Thread Jan Kundrát
jkt created this revision. jkt added a reviewer: broulik. jkt set the repository for this revision to rPOWERDEVIL Powerdevil. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY rPOWERDEVIL Powerdevil REVISION DETAIL https://phab

[Differential] [Request, 2 lines] D2658: Fix build failure introduced by the previous commit

2016-09-04 Thread Jan Kundrát
jkt created this revision. jkt added a reviewer: broulik. jkt set the repository for this revision to rPOWERDEVIL Powerdevil. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY My compiler complains about an attempted conve

Jenkins-kde-ci: plasma-tests master kf5-qt5 » Linux,gcc - Build # 193 - Unstable!

2016-09-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-tests%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/193/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 04 Sep 2016 21:58:11 + Build duration: 3 min 21 sec CHANGE SET No changes JUNIT RESULTS Name:

Re: Review Request 128815: Fix themes list with SDDM 0.14

2016-09-04 Thread Antonio Rojas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128815/ --- (Updated Sept. 4, 2016, 11:51 p.m.) Status -- This change has been m

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 297 - Still Unstable!

2016-09-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/297/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 04 Sep 2016 21:36:15 + Build duration: 7 min 41 sec CHANGE SET Revision ee2577125c5ac50ed21e828eb09e63

[Differential] [Closed] D2654: AppEntry: load icon on demand, to speed up plasma startup

2016-09-04 Thread dfaure (David Faure)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOPee2577125c5a: AppEntry: load icon on demand, to speed up plasma startup (authored by dfaure). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.or

[Differential] [Closed] D2645: make sure the vertical systary size hints are fixed

2016-09-04 Thread mart (Marco Martin)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACE7b7c680f3a4a: make sure the vertical systary size hints are fixed (authored by mart). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D2645?vs=6408&id=6430#toc REPOSITORY rPLASMAWOR

[Differential] [Commented On] D2652: Let the application control the menu further

2016-09-04 Thread mart (Marco Martin)
mart added inline comments. INLINE COMMENTS > GlobalDrawer.qml:184 > + */ > +property bool moveUpOnLeafAction: true > + resetMenuOnTriggered? resetOnActionTriggered? REPOSITORY rKIRIGAMI Kirigami REVISION DETAIL https://phabricator.kde.org/D2652 EMAIL PREFERENCES https://phabric

[Differential] [Commented On] D2652: Let the application control the menu further

2016-09-04 Thread colomar (Thomas Pfeiffer)
colomar added a comment. To provide some context: Discover now switched to use the sidebar as a navigation menu, controlling which category of applications to show in the main window. If clicking a menu item went back to the top level, the menu would not correspond to what's shown anymore

[Differential] [Request, 104 lines] D2656: [Kickoff] Lazy load more

2016-09-04 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, hein. 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 Neither the apps model no

[Differential] [Accepted] D2654: AppEntry: load icon on demand, to speed up plasma startup

2016-09-04 Thread broulik (Kai Uwe Broulik)
broulik accepted this revision. This revision is now accepted and ready to land. REPOSITORY rPLASMADESKTOP Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D2654 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: dfaure, broulik C

Re: Review Request 128819: grid layout for windowdecoration KCM

2016-09-04 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128819/ --- (Updated Sept. 4, 2016, 6:37 nachm.) Review request for kwin and Plasma.

[kio-extras] [Bug 366795] Paste action disabled when copying a local file to smb:// or mtp://

2016-09-04 Thread Emmanuel Pescosta via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=366795 Emmanuel Pescosta changed: What|Removed |Added CC||germano.massu...@gmail.com --- Comment #20

[Differential] [Request, 21 lines] D2654: AppEntry: load icon on demand, to speed up plasma startup

2016-09-04 Thread dfaure (David Faure)
dfaure created this revision. dfaure added a reviewer: broulik. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. TEST PLAN Starting plasmashell. It still takes too long, but now it's in QML stuff ;) REPOSITORY rPLASMADESKTOP Plasma Deskt

[Differential] [Request, 25 lines] D2652: Let the application control the menu further

2016-09-04 Thread apol (Aleix Pol Gonzalez)
apol created this revision. apol added reviewers: Kirigami, mart. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Introduces a property so that triggering leaf categories doesn't change the state of the menu. Introdu

[Differential] [Updated, 29 lines] D2033: Clear idle timeouts when session becomes inactive

2016-09-04 Thread broulik (Kai Uwe Broulik)
broulik updated this revision to Diff 6418. broulik added a comment. Cleanup patch a bit REPOSITORY rPOWERDEVIL Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2033?vs=4822&id=6418 REVISION DETAIL https://phabricator.kde.org/D2033 AFFECTED FILES daemon/powerdevilco

[Powerdevil] [Bug 353032] Add ability to set screen brightness for non-laptop displays so I can adjust to the room's lighting

2016-09-04 Thread Kai Uwe Broulik via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=353032 Kai Uwe Broulik changed: What|Removed |Added CC||k...@privat.broulik.de --- Comment #2 from Ka

[Differential] [Updated, 50 lines] D2534: [Desktop View] Place Plasma sidebars at the right screen edge in right-to-left mode

2016-09-04 Thread broulik (Kai Uwe Broulik)
broulik updated this revision to Diff 6417. broulik added a comment. Fix bindings and cleanup a bit REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2534?vs=6154&id=6417 REVISION DETAIL https://phabricator.kde.org/D2534 AFFECTED FILES d

[Differential] [Accepted] D2645: make sure the vertical systary size hints are fixed

2016-09-04 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. davidedmundson added a comment. This revision is now accepted and ready to land. You don't need to guess, the code is open source... Here is the reason. panel/main.qml the container has:

[Differential] [Accepted] D2648: [Runners KCM] Fix painting delegates in right-to-left layout

2016-09-04 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2648 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik,

[Differential] [Commented On] D2645: make sure the vertical systary size hints are fixed

2016-09-04 Thread mart (Marco Martin)
mart added a comment. In https://phabricator.kde.org/D2645#49268, @davidedmundson wrote: > Yes I can reproduce the bug. And I even downloaded this and confirm that then i can't reproduce it. > > But that doesn't answer my question. the hints of the applet seems sane. minimum

[Differential] [Commented On] D2534: [Desktop View] Place Plasma sidebars at the right screen edge in right-to-left mode

2016-09-04 Thread broulik (Kai Uwe Broulik)
broulik added a comment. The height of the dialog updates just fine (it's on the mainItem and not dialog itself) with the binding but indeed the X and Y binding just break at some point. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2534 EMAIL PR

Re: Review Request 128819: grid layout for windowdecoration KCM

2016-09-04 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128819/#review98861 --- good direction, commenting on some QML code style issues kc

[Differential] [Request, 13 lines] D2648: [Runners KCM] Fix painting delegates in right-to-left layout

2016-09-04 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, safaalfulaij. broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. TEST PLAN kcmshell5 plasmasearch

Re: Copy Activity - Create Activity by template?

2016-09-04 Thread Marco Martin
On Sat, Sep 3, 2016 at 2:18 PM, Michail Vourlakos wrote: > Hello everyone, > > I received from one my users a question: > > "I like your plasmoid but I want to use it with many of my activities but I > dont want its defaults, is there a way to set its defaults once and be kept > in ordet to be use

[Differential] [Commented On] D2645: make sure the vertical systary size hints are fixed

2016-09-04 Thread davidedmundson (David Edmundson)
davidedmundson added a comment. Yes I can reproduce the bug. And I even downloaded this and confirm that then i can't reproduce it. But that doesn't answer my question. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2645 EMAIL PREFERENCES

Re: Is it possible to know the panel's anchoring?

2016-09-04 Thread Marco Martin
On Sat, Sep 3, 2016 at 1:28 PM, Michail Vourlakos wrote: > Kai, I dont think so... > panel.location describes where the panel is placed, > bottom, top, left, right. Not how it is anchored, > > for example a panel is placed at left but is anchored at > bottom... The bottom value can not be accessed

[Differential] [Commented On] D2645: make sure the vertical systary size hints are fixed

2016-09-04 Thread mart (Marco Martin)
mart added a comment. In https://phabricator.kde.org/D2645#49198, @davidedmundson wrote: > Why is it needed? > > the panel containment is a layout - the system tray's current only size hint is minimum and it doesn't have a Layout.fillHeight - which means any other item which does hav

Re: Review Request 128815: Fix themes list with SDDM 0.14

2016-09-04 Thread David Edmundson
> On Sept. 2, 2016, 6:17 p.m., Fabian Vogt wrote: > > IMO you should use the same method as sddm, which is hardcoded at configure > > time: "@DATA_INSTALL_DIR@/themes". > > Wolfgang Bauer wrote: > I agree. > This change will still make it use ~/.local/share/sddm/themes/ if it > should

Re: Review Request 128815: Fix themes list with SDDM 0.14

2016-09-04 Thread Antonio Rojas
> On Sept. 2, 2016, 6:17 p.m., Fabian Vogt wrote: > > IMO you should use the same method as sddm, which is hardcoded at configure > > time: "@DATA_INSTALL_DIR@/themes". > > Wolfgang Bauer wrote: > I agree. > This change will still make it use ~/.local/share/sddm/themes/ if it > should