[Differential] [Closed] D2850: Prevent "TypeError: Cannot read property 'Layout' of null" warning on plasmashell startup

2016-09-24 Thread eliasp (Elias Probst)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP8bfd1e94a57d: Prevent "TypeError: Cannot read property 'Layout' of null" warning on… (authored by eliasp). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabr

Jenkins-kde-ci: plasma-desktop Plasma-5.8 stable-kf5-qt5 » Linux,gcc - Build # 17 - Failure!

2016-09-24 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.8%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/17/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 24 Sep 2016 12:34:02 + Build duration: 1 min 54 sec CHANGE SET Revision c9830d0ac6e55355161d3

[Differential] [Closed] D2849: warnings--

2016-09-24 Thread davidedmundson (David Edmundson)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACEdc1e6ae26073: warnings-- (authored by davidedmundson). REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2849?vs=6900&id=6911 REVISION

[Differential] [Closed] D2846: Let kicker display executable items instead of empty buttons.

2016-09-24 Thread ochurlaud (Olivier Churlaud)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP5c966a42554f: Let kicker display executable items instead of empty buttons. (authored by ochurlaud). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator

[Differential] [Accepted] D2846: Let kicker display executable items instead of empty buttons.

2016-09-24 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2846 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: ochurlaud, hein, #plasma, davidedmundson, mart Cc: mart,

[Differential] [Accepted] D2849: warnings--

2016-09-24 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY rPLASMAWORKSPACE Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D2849 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailprefe

[Differential] [Accepted] D2846: Let kicker display executable items instead of empty buttons.

2016-09-24 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2846 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: ochurlaud, hein, #plasma, d

[Differential] [Updated, 5 lines] D2846: Let kicker display executable items instead of empty buttons.

2016-09-24 Thread ochurlaud (Olivier Churlaud)
ochurlaud updated this revision to Diff 6909. ochurlaud added a comment. Use !contains() instead of index() == -1 REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2846?vs=6906&id=6909 BRANCH master REVISION DETAIL https://phabricator.kd

[Differential] [Accepted] D2850: Prevent "TypeError: Cannot read property 'Layout' of null" warning on plasmashell startup

2016-09-24 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. Me neither :( REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2850 EMAIL PREFEREN

[Differential] [Updated, 5 lines] D2846: Let kicker display executable items instead of empty buttons.

2016-09-24 Thread ochurlaud (Olivier Churlaud)
ochurlaud updated this revision to Diff 6906. ochurlaud added a comment. Check for type, as asked by Sho REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2846?vs=6896&id=6906 BRANCH master REVISION DETAIL https://phabricator.kde.org/D28

[Differential] [Commented On] D2850: Prevent "TypeError: Cannot read property 'Layout' of null" warning on plasmashell startup

2016-09-24 Thread eliasp (Elias Probst)
eliasp added inline comments. INLINE COMMENTS > davidedmundson wrote in Panel.qml:78 > in theory, this line should be doing that. > > Any idea why it isn't? Unfortunately - no idea why it doesn't prevent that. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kd

[Differential] [Changed Subscribers] D2850: Prevent "TypeError: Cannot read property 'Layout' of null" warning on plasmashell startup

2016-09-24 Thread davidedmundson (David Edmundson)
davidedmundson added inline comments. INLINE COMMENTS > Panel.qml:78 > property: "length" > when: containment > value: { in theory, this line should be doing that. Any idea why it isn't? REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricato

[Differential] [Closed] D2847: Use correct parent for folderview context menu actions

2016-09-24 Thread davidedmundson (David Edmundson)
davidedmundson closed this revision. davidedmundson added a comment. Submitted as https://phabricator.kde.org/rPLASMADESKTOPc8d3546a9e3ea1dfdb1853a33172824e7ff46feb REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2847 EMAIL PREFERENCES https://pha

[Differential] [Closed] D2841: Don't load system apps model twice

2016-09-24 Thread davidedmundson (David Edmundson)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP5fe0c2398218: Don't load system apps model twice (authored by davidedmundson). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2841?vs=6886

[Differential] [Closed] D2842: Add kinfocenter to the computer tab of Kickoff

2016-09-24 Thread davidedmundson (David Edmundson)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP94a94548418e: Add kinfocenter to the computer tab of Kickoff (authored by davidedmundson). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 369 - Fixed!

2016-09-24 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/369/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 24 Sep 2016 10:16:36 + Build duration: 5 min 47 sec CHANGE SET Revision f7bba1ce7486ea7adc966854faa6167

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 369 - Fixed!

2016-09-24 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/369/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 24 Sep 2016 10:16:36 + Build duration: 5 min 47 sec CHANGE SET Revision f7bba1ce7486ea7adc966854faa6167

[Differential] [Accepted] D2834: Prefer wallpapers with same aspect ratio as screen

2016-09-24 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 BRANCH Plasma/5.8 REVISION DETAIL https://phabricator.kde.org/D2834 EMAIL PREFERENCES https://phabricator

[Differential] [Closed] D2848: Fix binding loop in pager

2016-09-24 Thread davidedmundson (David Edmundson)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOPa544bd5c0ee8: Fix binding loop in pager (authored by davidedmundson). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2848?vs=6898&id=6903

[Differential] [Accepted] D2848: Fix binding loop in pager

2016-09-24 Thread hein (Eike Hein)
hein accepted this revision. hein added a reviewer: hein. hein added a comment. This revision is now accepted and ready to land. Thanks! REPOSITORY rPLASMADESKTOP Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D2848 EMAIL PREFERENCES https://phabricator.kde

[Differential] [Accepted] D2847: Use correct parent for folderview context menu actions

2016-09-24 Thread hein (Eike Hein)
hein accepted this revision. hein added a reviewer: hein. This revision is now accepted and ready to land. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D2847 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: fvogt, #

[Differential] [Requested Changes To] D2846: Let kicker display executable items instead of empty buttons.

2016-09-24 Thread hein (Eike Hein)
hein requested changes to this revision. hein added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > recentusagemodel.cpp:175 > > -if (!service || !service->isApplication()) { > +if (!service || service->exec().isEmpty()) { > return QVariant();

[Differential] [Accepted] D2841: Don't load system apps model twice

2016-09-24 Thread hein (Eike Hein)
hein accepted this revision. hein added a reviewer: hein. hein added a comment. IIRC there was a reason for this related to migrating from the legacy config, where stuff had to be merged ... but since I didn't add a code comment to the effect, oh well, let's try I guess. REPOSITORY rPLASMA

[Differential] [Accepted] D2842: Add kinfocenter to the computer tab of Kickoff

2016-09-24 Thread hein (Eike Hein)
hein accepted this revision. hein added a reviewer: hein. REPOSITORY rPLASMADESKTOP Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D2842 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, mart, #plasma, hein Cc:

Re: Review Request 106911: ksmserver should ignore non-executable files under ~/.kde4/Autostart

2016-09-24 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/106911/#review99497 --- This is for kde-workspace which is now deprecated. Closing.

Re: Review Request 106911: ksmserver should ignore non-executable files under ~/.kde4/Autostart

2016-09-24 Thread Jekyll Wu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/106911/ --- (Updated Sept. 24, 2016, 10:04 a.m.) Status -- This change has been

Re: Review Request 129009: [busy indicator] Limit animation loops

2016-09-24 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129009/#review99496 --- I don't think we can try and "fix" application bugs in a libr

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 368 - Failure!

2016-09-24 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/368/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 24 Sep 2016 09:52:00 + Build duration: 45 sec CHANGE SET Revision a7b5856387ce35d132e61436b876d7120b4b0

[Differential] [Request, 3 lines] D2850: Prevent "TypeError: Cannot read property 'Layout' of null" warning on plasmashell startup

2016-09-24 Thread eliasp (Elias Probst)
eliasp created this revision. eliasp added a reviewer: Plasma. eliasp set the repository for this revision to rPLASMADESKTOP Plasma Desktop. eliasp added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY When starting `plasmashell`, the following warnin