[Powerdevil] [Bug 377513] New: Wayland sometimes doesn't recover from standby

2017-03-11 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=377513 Bug ID: 377513 Summary: Wayland sometimes doesn't recover from standby Product: Powerdevil Version: 5.9.3 Platform: Other OS: Linux Status: UNCONFIRMED Severity:

D5013: Truncate url string in KIO::job description to avoid freezing up plasma notification

2017-03-11 Thread Yuen Hoe Lim
yuenlim updated this revision to Diff 12403. yuenlim added a comment. Something like this? REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5013?vs=12391&id=12403 REVISION DETAIL https://phabricator.kde.org/D5013 AFFECTED FILES src/core/job.cpp src/core/j

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

2017-03-11 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/665/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 12 Mar 2017 02:25:50 + Build duration: 14 min CHANGE SET Revision a577488ffafbe4ea5068dfc6ca91bb3848cb

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

2017-03-11 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/664/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 12 Mar 2017 02:07:19 + Build duration: 16 min CHANGE SET Revision c090c59513db15757182f97ad2f253d569fa

D5013: Truncate url string in KIO::job description to avoid freezing up plasma notification

2017-03-11 Thread David Faure
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Extra credit for factorizing this into a file-static function and handling data: URLs separately (no point in seeing 100 chars of data crap, we could do data:[...] or something like that).

Re: LGPL a possibility for Breeze Qt widget style?

2017-03-11 Thread Elvis Stansvik
2017-03-11 17:41 GMT+01:00 Christoph Feck : > On 11.03.2017 16:44, Elvis Stansvik wrote: >> >> 2017-03-10 23:26 GMT+01:00 Elvis Stansvik : >>> >>> In the end, I realized it wasn't really feasible to bundle Breeze >>> inside the AppImage anyway, because of the many KF5 dependencies. I >>> thought th

D4970: Get rid of QWeakPointer

2017-03-11 Thread Denis Shienkov
denisshienkov added a comment. Can I get a "commit access"? If yes, then how? I have read many WiKi's pages about KDE commit policy and so on.. But I did not understand how it works with KDE? Could you please recommend for me some appropriate wiki? PS: I know how to work with Qt's JIRA,

D5013: Truncate url string in KIO::job description to avoid freezing up plasma notification

2017-03-11 Thread Yuen Hoe Lim
yuenlim updated this revision to Diff 12391. yuenlim added a comment. Use KStringHandler::csqueeze for truncating! REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5013?vs=12380&id=12391 REVISION DETAIL https://phabricator.kde.org/D5013 AFFECTED FILES src/c

Review Request 130006: Avoid showing wrong timezone suffix in krunner time

2017-03-11 Thread Ronnie Thomas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130006/ --- Review request for Plasma. Repository: kdeplasma-addons Description ---

D4970: Get rid of QWeakPointer

2017-03-11 Thread Albert Astals Cid
aacid resigned from this revision. aacid added a comment. This revision is now accepted and ready to land. Ok, abstaining. David you should commit it since AFAICS Denis does not have commit access. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D4970 To: deni

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
This revision was automatically updated to reflect the committed changes. Closed by commit R122:c6b76da7a679: Add UI for hybrid suspend mode (authored by strobach). REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5016?vs=12388&id=12390 REVISION DETAIL htt

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
strobach added a comment. In https://phabricator.kde.org/D5016#94363, @broulik wrote: > The question is whether we should treat "hybrid suspend" and "suspend" as separate options or just have a global checkbox "[X] When possible use hybrid suspend". I don't think so. I do, for in

D4970: Get rid of QWeakPointer

2017-03-11 Thread Christoph Feck
cfeck set the repository for this revision to R122 Powerdevil. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D4970 To: denisshienkov, afiestas, ltoscano, volkov, davidedmundson, aacid Cc: aacid, davidedmundson, plasma-devel, volkov, progwolff, lesliezhai, ali-mohame

Re: LGPL a possibility for Breeze Qt widget style?

2017-03-11 Thread Christoph Feck
On 11.03.2017 16:44, Elvis Stansvik wrote: 2017-03-10 23:26 GMT+01:00 Elvis Stansvik : In the end, I realized it wasn't really feasible to bundle Breeze inside the AppImage anyway, because of the many KF5 dependencies. I thought that the widget style .so would only have Qt as dependency, but rea

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread Kai Uwe Broulik
broulik accepted this revision. broulik added a comment. This revision is now accepted and ready to land. Thanks a lot! Interestingly enough PowerDevil's suspend session action already had a case for hybrid suspend it just wasn't exposed in the UI. The question is whether we should t

D4970: Get rid of QWeakPointer

2017-03-11 Thread Albert Astals Cid
aacid requested changes to this revision. aacid added a comment. This revision now requires changes to proceed. What repository is this patch for? REVISION DETAIL https://phabricator.kde.org/D4970 To: denisshienkov, afiestas, ltoscano, volkov, davidedmundson, aacid Cc: aacid, davidedmundson

Re: LGPL a possibility for Breeze Qt widget style?

2017-03-11 Thread Elvis Stansvik
2017-03-10 23:26 GMT+01:00 Elvis Stansvik : > 2017-03-09 19:21 GMT+01:00 Elvis Stansvik : >> 2017-03-09 18:41 GMT+01:00 Jonathan Riddell : >>> On Thu, Mar 09, 2017 at 06:30:36PM +0100, Elvis Stansvik wrote: I just wish the FSF FAQ was more clear on this (direct dynamic linking vs dlopen()

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
strobach added reviewers: broulik, sebas. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D5016 To: strobach, broulik, sebas Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
strobach created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Add "Hybrid suspend" option to suspend actions and button handler actions. The action is already supported by Solid. REPOSITORY R122 Powe

D5013: Truncate url string in KIO::job description to avoid freezing up plasma notification

2017-03-11 Thread David Faure
dfaure added a comment. Can you use KStringHandler::csqueeze ? This way filenames are still visible. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D5013 To: yuenlim, #plasma Cc: dfaure, plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abet

D5013: Truncate url string in KIO::job description to avoid freezing up plasma notification

2017-03-11 Thread Yuen Hoe Lim
yuenlim created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY When posting large raw image data to imgur in QuickShare, the KIO job ends up triggering a notification that displays th