[Differential] [Request, 2 lines] D4342: Use texture atlas for static icon item

2017-01-30 Thread David Edmundson
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY Icon's are generally small, they may as well go in the existing shared

[Differential] [Accepted] D4342: Use texture atlas for static icon item

2017-01-30 Thread Marco Martin
mart accepted this revision. mart added a reviewer: mart. mart added a comment. This revision is now accepted and ready to land. thatnks, makes sense (and let's see if makes a difference with the phone, may be telling) REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION D

[Differential] [Closed] D4342: Use texture atlas for static icon item

2017-01-30 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R242:8d60a8f1f5b9: Use texture atlas for static icon item (authored by davidedmundson). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4342?vs=10695&i

[Differential] [Request, 4 lines] D4348: [Containment Interface] No need for values() as contains() looks up keys

2017-01-30 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to R242 Plasma Framework (Library). Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY Avoids

[Differential] [Accepted] D4348: [Containment Interface] No need for values() as contains() looks up keys

2017-01-30 Thread David Edmundson
davidedmundson accepted this revision. davidedmundson added a reviewer: davidedmundson. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4348 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel

[Differential] [Request, 4 lines] D4350: [FrameData] Avoid iterating keys()

2017-01-30 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to R242 Plasma Framework (Library). Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY Instea

[Differential] [Accepted] D4350: [FrameData] Avoid iterating keys()

2017-01-30 Thread Marco Martin
mart accepted this revision. mart added a reviewer: mart. mart added a comment. This revision is now accepted and ready to land. it would also be interesting to see why the destructor is being called so many times at startup (some model somewhere emitting resets that shouldn't?).. as at start

[Differential] [Closed] D4348: [Containment Interface] No need for values() as contains() looks up keys

2017-01-30 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:fe3990556046: [Containment Interface] No need for values() as contains() looks up keys (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricat

[Differential] [Closed] D4350: [FrameData] Avoid iterating keys()

2017-01-30 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:d38be811e82b: [FrameData] Avoid iterating keys() (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4350?vs=10719&id=10721 RE

[Differential] [Changed Subscribers] D4348: [Containment Interface] No need for values() as contains() looks up keys

2017-01-30 Thread Aleix Pol Gonzalez
apol added inline comments. INLINE COMMENTS > containmentinterface.cpp:565 > QPoint posi; // will be overwritten with the event's position > -if (m_dropPoints.keys().contains(tjob)) { > -posi = m_dropPoints[tjob]; > +if (m_dropPoints.contains(tjob)) { > +

[Differential] [Request, 3 lines] D4352: [ContainmentInterface] Use setSize instead of setWidth/setHeight in createApplet

2017-01-30 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to R242 Plasma Framework (Library). Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY This a

[Differential] [Updated] D4351: [QuickViewSharedEngine] Use setSize instead of setWidth/setHeight

2017-01-30 Thread Kai Uwe Broulik
broulik updated the summary for this revision. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D4351 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma Cc: plasma-devel, #frameworks, lesliezhai, ali-mohamed, jensreu

[Differential] [Request, 9 lines] D4351: [QuickViewSharedEngine] Use setSize instead of setWidth/setHeight

2017-01-30 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to R296 KDeclarative. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY This avoids needless

[Differential] [Accepted] D4351: [QuickViewSharedEngine] Use setSize instead of setWidth/setHeight

2017-01-30 Thread Marco Martin
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D4351 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, ma

[Differential] [Accepted] D4352: [ContainmentInterface] Use setSize instead of setWidth/setHeight in createApplet

2017-01-30 Thread Marco Martin
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4352 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: brouli

[Differential] [Closed] D4352: [ContainmentInterface] Use setSize instead of setWidth/setHeight in createApplet

2017-01-30 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:031bd773b7c2: [ContainmentInterface] Use setSize instead of setWidth/setHeight in createApplet (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://p

[Differential] [Closed] D4351: [QuickViewSharedEngine] Use setSize instead of setWidth/setHeight

2017-01-30 Thread Kai Uwe Broulik
broulik closed this revision. broulik added a comment. Committed as https://cgit.kde.org/kdeclarative.git/commit/?id=2220a46702e670b5758892902797241d9b300bcf REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D4351 EMAIL PREFERENCES https://phabricator.kde.org/se

Re: Review Request 129833: KToolTipWidget: hide immediately if content gets destroyed

2017-01-30 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129833/ --- (Updated Jan. 30, 2017, 11:08 p.m.) Status -- This change has been m

Re: Review Request 129849: KToolTipWidget: don't take ownership of the content widget

2017-01-30 Thread Elvis Angelaccio
> On Jan. 29, 2017, 10:05 p.m., Aleix Pol Gonzalez wrote: > > Shouldn't the tooltip be the parent of the widget anyway? I'd say this is up for discussion, since the api is fairly new. Either we make the tooltip take ownership and we warn about this (to avoid double delete crashes), or we don't

Re: Review Request 129694: About dialog: fix scrolling in Authors tab

2017-01-30 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129694/ --- (Updated Jan. 30, 2017, 11:26 p.m.) Status -- This change has been d

[Differential] [Request, 87 lines] D4355: Add methods to hide the reveal password action

2017-01-30 Thread Elvis Angelaccio
elvisangelaccio created this revision. elvisangelaccio added reviewers: cfeck, broulik. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY The new `lineedit_reveal_password` kiosk key cannot be queried from kwidgetsaddons

[Differential] [Updated] D4355: Add methods to hide the reveal password action

2017-01-30 Thread Elvis Angelaccio
elvisangelaccio updated the summary for this revision. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D4355 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: elvisangelaccio, cfeck, broulik Cc: #frameworks

Raising the requirement to Qt 5.6 ?

2017-01-30 Thread Albert Astals Cid
Now that Qt 5.8 is out, should we increase requirement to Qt 5.6 as the rules say? Cheers, Albert

Re: Raising the requirement to Qt 5.6 ?

2017-01-30 Thread Aleix Pol
On Tue, Jan 31, 2017 at 12:59 AM, Albert Astals Cid wrote: > Now that Qt 5.8 is out, should we increase requirement to Qt 5.6 as the rules > say? +1 Let's!

[Differential] [Commented On] D4351: [QuickViewSharedEngine] Use setSize instead of setWidth/setHeight

2017-01-30 Thread Eike Hein
hein added a comment. You have no idea how much I love you right now! This has annoyed me so long. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D4351 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, mart C

[Differential] [Request, 5 lines] D4357: Re-enable Log-to-File for KNotifications

2017-01-30 Thread Christoph Feck
cfeck created this revision. cfeck set the repository for this revision to R289 KNotifications. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D4357 AFFECTED

[Differential] [Updated, 5 lines] D4357: Re-enable Log-to-File for KNotifications

2017-01-30 Thread Christoph Feck
cfeck removed R289 KNotifications as the repository for this revision. cfeck updated this revision to Diff 10750. CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4357?vs=10749&id=10750 REVISION DETAIL https://phabricator.kde.org/D4357 AFFECTED FILES src/CMakeLists.txt src/knotific

[Differential] [Updated] D4357: Re-enable Log-to-File for KNotifications

2017-01-30 Thread Christoph Feck
cfeck updated the summary for this revision. cfeck updated the test plan for this revision. cfeck added a reviewer: mck182. cfeck set the repository for this revision to R289 KNotifications. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D4357 EMAIL PREFERENCES

Jenkins-kde-ci: threadweaver master stable-kf5-qt5 » Linux,gcc - Build # 122 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/threadweaver%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/122/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 01:05:15 + Build duration: 1 min 12 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: threadweaver master stable-kf5-qt5 » Linux,gcc - Build # 122 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/threadweaver%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/122/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 01:05:15 + Build duration: 1 min 12 sec CHANGE SET No changes JUNIT RESULTS

[Differential] [Commented On] D4335: Fix endianness issues once and for all in Blowfish algorithm

2017-01-30 Thread Michael Pyne
mpyne added a comment. When reviewing this, please see also the review chain at https://git.reviewboard.kde.org/r/127833/#review95717 , which had proposed removing the then-needless KWallet includes. In essence the sticking point we have currently is that fixing the Blowfish implementat

Jenkins-kde-ci: kwidgetsaddons master kf5-qt5 » Linux,gcc - Build # 180 - Failure!

2017-01-30 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/kwidgetsaddons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/180/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 02:07:56 + Build duration: 14 min CHANGE SET No changes

Jenkins-kde-ci: kwindowsystem master stable-kf5-qt5 » Linux,NoX11,gcc - Build # 143 - Unstable!

2017-01-30 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kwindowsystem%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/143/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Tue, 31 Jan 2017 02:45:14 + Build duration: 59 sec CHANGE SET No chan

Jenkins-kde-ci: kwidgetsaddons master kf5-qt5 » Linux,gcc - Build # 181 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kwidgetsaddons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/181/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 03:12:45 + Build duration: 1 min 58 sec CHANGE SET No changes JUNIT RESULTS Name

Jenkins-kde-ci: kwidgetsaddons master kf5-qt5 » Linux,gcc - Build # 181 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kwidgetsaddons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/181/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 03:12:45 + Build duration: 1 min 58 sec CHANGE SET No changes JUNIT RESULTS Name

Jenkins-kde-ci: kwindowsystem master kf5-qt5 » Linux,All,gcc - Build # 145 - Unstable!

2017-01-30 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kwindowsystem%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/145/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Tue, 31 Jan 2017 03:25:17 + Build duration: 1 min 53 sec CHANGE SET No changes

Jenkins-kde-ci: kwindowsystem master kf5-qt5 » Linux,NoX11,gcc - Build # 145 - Unstable!

2017-01-30 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kwindowsystem%20master%20kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/145/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Tue, 31 Jan 2017 03:25:17 + Build duration: 1 min 2 sec CHANGE SET No change

Jenkins-kde-ci: kdoctools master stable-kf5-qt5 » Linux,gcc - Build # 195 - Failure!

2017-01-30 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/kdoctools%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/195/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 03:59:29 + Build duration: 23 sec CHANGE SET No changes

Jenkins-kde-ci: kdoctools master stable-kf5-qt5 » Linux,gcc - Build # 196 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kdoctools%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/196/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 04:31:50 + Build duration: 48 sec CHANGE SET No changes JUNIT RESULTS Name: (r

Jenkins-kde-ci: kdoctools master stable-kf5-qt5 » Linux,gcc - Build # 196 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kdoctools%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/196/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 04:31:50 + Build duration: 48 sec CHANGE SET No changes JUNIT RESULTS Name: (r

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 404 - Unstable!

2017-01-30 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/404/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 04:33:05 + Build duration: 15 min CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 405 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/405/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 05:42:12 + Build duration: 16 min CHANGE SET No changes JUNIT RESULTS Name: (root) F

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 405 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/405/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 05:42:12 + Build duration: 16 min CHANGE SET No changes JUNIT RESULTS Name: (root) F

Jenkins-kde-ci: kactivities master stable-kf5-qt5 » Linux,gcc - Build # 417 - Failure!

2017-01-30 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/kactivities%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/417/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 06:19:27 + Build duration: 3 min 8 sec CHANGE SET No changes

Jenkins-kde-ci: kactivities master stable-kf5-qt5 » Linux,gcc - Build # 418 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kactivities%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/418/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 06:23:38 + Build duration: 4 min 42 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: kactivities master stable-kf5-qt5 » Linux,gcc - Build # 418 - Fixed!

2017-01-30 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kactivities%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/418/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 31 Jan 2017 06:23:38 + Build duration: 4 min 42 sec CHANGE SET No changes JUNIT RESULTS