[Powerdevil] [Bug 376755] Screen locker is delayed, exposing data

2017-02-21 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=376755 Kai Uwe Broulik changed: What|Removed |Added CC||k...@privat.broulik.de --- Comment #9 from Ka

[Differential] [Accepted] D4705: Don't assume initialized if there are pending dbus replies

2017-02-21 Thread Jan Grulich
jgrulich accepted this revision. jgrulich added a comment. This revision is now accepted and ready to land. Ok, I see that this can probably happen when switching connections before secrets are actually received. Thanks for the patch!!. REPOSITORY R116 Plasma Network Management Applet BRAN

[Powerdevil] [Bug 376755] Screen locker is delayed, exposing data

2017-02-21 Thread Martin Gräßlin
https://bugs.kde.org/show_bug.cgi?id=376755 Martin Gräßlin changed: What|Removed |Added Product|kscreenlocker |Powerdevil Assignee|plasma-b...@kde.

[Differential] [Request, 15 lines] D4713: Fix binding loop regression in FrameSVGItem

2017-02-21 Thread David Edmundson
davidedmundson created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY https://phabricator.kde.org/R242:d8a1a9eb084b19e552c789244267f7346e1b27a8 introduces an unintended code change, resizeFram

Re: Review Request 129917: Add a cache monitor to the System Load Viewer applet

2017-02-21 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129917/#review102571 --- Ship it! Ship It! - David Edmundson On Feb. 7, 2017, 9

[Differential] [Commented On] D4491: Let make taskmanager tooltip readable again

2017-02-21 Thread Mark Gaiser
markg added a comment. In https://phabricator.kde.org/D4491#88362, @anthonyfieroni wrote: > I must investigate why elide does not work, margins are a bit different from others like systray, kicker, etc. > I figure out it, maximumLineCount or height sould be setted, i unsetted height..

Re: Plasma Wayland relocatability

2017-02-21 Thread Martin Gräßlin
Am 2017-02-21 12:48, schrieb Harald Sitter: # startplasmacompositor - Resolves qdbus via qtpaths which is not relocatable, that's not really our fault and probably not too easy to solve or even worth it. Finding qdbus reliably has been a long standing problem unfortunately. - Hard-compiles the

[Differential] [Accepted] D4701: [Greeter] Also clear selection clipboard

2017-02-21 Thread Martin Gräßlin
graesslin accepted this revision. This revision is now accepted and ready to land. REPOSITORY R133 KScreenLocker REVISION DETAIL https://phabricator.kde.org/D4701 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, graesslin Cc: plasma-deve

[Differential] [Request, 8 lines] D4712: Ungrab mouse on menu close

2017-02-21 Thread Anthony Fieroni
anthonyfieroni created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Same as plasma-framework, on audio volume is weird as hell. Right click -> settings -> click in config window :) REPOSITORY R120 Pla

[Differential] [Request, 14 lines] D4711: Ungrab mouse on menu close

2017-02-21 Thread Anthony Fieroni
anthonyfieroni created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY Qt 5.8 has a bit changed and ungrabbing at show does not work. REPOSITORY R242 Plasma Framework (Library) REVI

[Differential] [Commented On] D4491: Let make taskmanager tooltip readable again

2017-02-21 Thread Anthony Fieroni
anthonyfieroni added a comment. I must invetigate why elide does not work, margins are a bit different from others like systray, kicker, etc. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4491 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/e

Re: Review Request 129917: Add a cache monitor to the System Load Viewer applet

2017-02-21 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129917/#review102567 --- Sebas can this be commited now or you have more concerns? -

[Maniphest] [Commented On] T5412: Improve Plasma PA usability

2017-02-21 Thread Roman Gilg
subdiff added a comment. > This needs to go in, I forgot about it: https://phabricator.kde.org/D2314 Looks good! As Thomas said in the review though, I'm also in favor of offering at least the "Default" setting directly in the applet or add a button next to every stream to show this cont

[Differential] [Updated, 57 lines] D4550: Task Manager: Make the icon size configurable for task buttons in a vertical task manager

2017-02-21 Thread Peter Palaga
ppalaga updated this revision to Diff 11596. ppalaga added a comment. In https://phabricator.kde.org/D4550#88247, @hein wrote: > I'm starting to like it, but it's a little bit scary. The thing is that we don't want to alter behavior in existing user setups, which is mostly narrow vertica

[Differential] [Closed] D4688: [FrameSvgItemMargins] Don't update on repaintNeeded

2017-02-21 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:408b31166227: [FrameSvgItemMargins] Don't update on repaintNeeded (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4688?vs=1

[Differential] [Request, 8 lines] D4709: [KSplashQML] Drop "Loading stage" debug output

2017-02-21 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY It is using QDateTime and secsTo which needs to take into potential timezone changes and what not. I don't think it's worth porting it to QE

[Differential] [Abandoned] D4362: RFC: [AppletQuickItem] Cache QQmlComponent used for settings QtQuick Controls 1 style

2017-02-21 Thread Kai Uwe Broulik
broulik abandoned this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4362 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma Cc: mart, davidedmundson, plasma-devel, #frameworks, progwolff,

[Differential] [Commented On] D4290: [Calculator Runner] Use "approximate" approximation mode

2017-02-21 Thread Kai Uwe Broulik
broulik added a comment. Ping REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D4290 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma_workspaces, mart Cc: mart, plasma-devel, progwolff, lesliezhai, ali-moha

[Differential] [Closed] D1771: [Workspace Options] Add option to disable OSD

2017-02-21 Thread Kai Uwe Broulik
broulik closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1771 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma:_design, graesslin, #plasma Cc: andreaska, colomar, graesslin, plasma-devel, p

[Differential] [Commented On] D4703: move the view at the correct index at startup

2017-02-21 Thread Marco Martin
mart added inline comments. INLINE COMMENTS > davidedmundson wrote in kcm.cpp:320 > We're doing the exact same logic here. Duplicating it without deleting this > is definitely wrong. > > It looks like this should work as we've loaded the model and the model is > populated at startup. > However

[Differential] [Updated, 21 lines] D4703: move the view at the correct index at startup

2017-02-21 Thread Marco Martin
mart updated this revision to Diff 11590. mart added a comment. Restricted Application edited projects, added Plasma; removed KWin. remove duplicate code REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4703?vs=11584&id=11590 BRANCH phab/selectIndex REVISION

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

2017-02-21 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/632/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 21 Feb 2017 14:57:18 + Build duration: 5 min 42 sec CHANGE SET Revision f10a6f4eba65e091a08aa47e6b87bb

[Differential] [Closed] D4706: Overhaul app associativity heuristic to give precendence to StartupWMClass.

2017-02-21 Thread Eike Hein
This revision was automatically updated to reflect the committed changes. Closed by commit R120:f17a395a40e9: Overhaul app associativity heuristic to give precendence to StartupWMClass. (authored by hein). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org

[Differential] [Accepted] D4706: Overhaul app associativity heuristic to give precendence to StartupWMClass.

2017-02-21 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D4706 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: hein, #plasma, dav

Re: KDM5?

2017-02-21 Thread Marco Martin
On Tuesday 21 February 2017, René J.V. Bertin wrote: > Hello, > > I get the impression I'm not the only one who hates the looks of SDDM (to > put it bluntly). How difficult/impossible would it be to port KDM, > presuming that hasn't been done already? Evidently I wouldn't expect > Wayland support.

[Differential] [Commented On] D4705: Don't assume initialized if there are pending dbus replies

2017-02-21 Thread Elvis Angelaccio
elvisangelaccio added a comment. In https://phabricator.kde.org/D4705#88274, @jgrulich wrote: > If secrets are requested then it should immediately return from initialize() and given that the kcm is not marked as initialized until we get reply. To me it appears that what you did shouldn'

[Differential] [Updated] D4706: Overhaul app associativity heuristic to give precendence to StartupWMClass.

2017-02-21 Thread Kai Uwe Broulik
broulik added a comment. Big +1, works beautifully: - Chrome windows behave like normal - Chrome app in Default user is treated like separate app - Chrome app in secondary Profile ("Profile_1" suffix) is treated like a separate app, too - LibreOffice Launcher is mapped fine - Libr

[Differential] [Updated] D4705: Don't assume initialized if there are pending dbus replies

2017-02-21 Thread Jan Grulich
jgrulich added a comment. If secrets are requested then it should immediately return from initialize() and given that the kcm is not marked as initialized until we get reply. To me it appears that what you did shouldn't be necessary. REPOSITORY R116 Plasma Network Management Applet REVISI

[Differential] [Request, 150 lines] D4706: Overhaul app associativity heuristic to give precendence to StartupWMClass.

2017-02-21 Thread Eike Hein
hein created this revision. Restricted Application added a project: Plasma. REVISION SUMMARY WM_CLASS instance and general classes are checked against StartupWMClass prior to most rewrite and mapping rules, and prior to the DesktopEntryName and Name checks. This matches the fd.o spec mo

[Differential] [Request, 8 lines] D4705: Don't assume initialized if there are pending dbus replies

2017-02-21 Thread Elvis Angelaccio
elvisangelaccio created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY ConnectionEditorBase::initialize() does not count the number of pending dbus replies, so it is currently assuming that if control rea

Re: KDM5?

2017-02-21 Thread Alexey Andreyev
https://github.com/Match-Yang/sddm-deepin https://github.com/absturztaube/sddm-archlinux-theme https://github.com/sniper1720/elegant-sddm-manjarolinux-theme https://github.com/AlfredoRamos/sddm-urbanlifestyle-theme 2017-02-21 15:04 GMT+03:00 René J.V. Bertin : > Hello, > > I get the impression I'

[Differential] [Commented On] D4550: Task Manager: Make the icon size configurable for task buttons in a vertical task manager

2017-02-21 Thread Eike Hein
hein added a comment. I'm starting to like it, but it's a little bit scary. The thing is that we don't want to alter behavior in existing user setups, which is mostly narrow vertical panels that don't get text. I fear that with default settings, due to lowering the threshold to 5 from 3, the

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

2017-02-21 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/631/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 21 Feb 2017 12:27:32 + Build duration: 7 min 27 sec CHANGE SET Revision 30388c7361889372cfdc9db77aeacc

[Differential] [Requested Changes] D4703: move the view at the correct index at startup

2017-02-21 Thread David Edmundson
davidedmundson requested changes to this revision. davidedmundson added a comment. This revision now requires changes to proceed. The QML hack is fine, but it seems like you've written the C++ part without reading any of the existing code. INLINE COMMENTS > kcm.cpp:320 > if (auto listVi

Re: Plasma Wayland relocatability

2017-02-21 Thread Harald Sitter
On Tue, Feb 21, 2017 at 12:48 PM, Harald Sitter wrote: > # ksmserver > > Starting here things seem to largely not be hardcoded anymore. Rejoice! Spoke too soon, kactivitymanagerd forks itself to detach... using its own installation path as hard compiled path as if it didn't know where it liv

Re: KDM5?

2017-02-21 Thread David Edmundson
If you hate the look, change the look locally. It supports themes. Changing the backend because of the look would be ridiculous. David

KDM5?

2017-02-21 Thread René J . V . Bertin
Hello, I get the impression I'm not the only one who hates the looks of SDDM (to put it bluntly). How difficult/impossible would it be to port KDM, presuming that hasn't been done already? Evidently I wouldn't expect Wayland support. R.

[Differential] [Request, 17 lines] D4703: move the view at the correct index at startup

2017-02-21 Thread Marco Martin
mart created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: KWin, kwin, plasma-devel. REVISION SUMMARY using the same trick as elsewhere, set the currentIndex and move the view to currentIndex right at startup the only way to be sure is

Plasma Wayland relocatability

2017-02-21 Thread Harald Sitter
Hola! For a while now I've been chipping away at making Plasma relocatable with the purpose of putting Plasma in a snap package (same applies to appimage FWIW, flatpak uses a static path), which happens to mount the bundle content under `/snap/$name/$REV/` meaning there is a severe need to be to h

[Differential] [Closed] D4702: IconItemTest: Fix loadPixmap and loadImage tests

2017-02-21 Thread David Rosca
This revision was automatically updated to reflect the committed changes. Closed by commit R242:148ff6311bfe: IconItemTest: Fix loadPixmap and loadImage tests (authored by drosca). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4702?vs=11582

[Differential] [Request, 2 lines] D4702: IconItemTest: Fix loadPixmap and loadImage tests

2017-02-21 Thread David Rosca
drosca created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY It only works on CI because data/test_image.png size is the same as implicitSize of IconItem (KIconLoader::Dialog). TES

[Differential] [Accepted] D4702: IconItemTest: Fix loadPixmap and loadImage tests

2017-02-21 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D4702 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca,

[Differential] [Closed] D4674: StreamRestore: Cache values in writeChanges until receiving change from pa

2017-02-21 Thread David Rosca
This revision was automatically updated to reflect the committed changes. Closed by commit R115:ba844b3e0e36: StreamRestore: Cache values in writeChanges until receiving change from pa (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabrica

[Differential] [Closed] D4675: Applet: Add configure button next to tabbar

2017-02-21 Thread David Rosca
This revision was automatically updated to reflect the committed changes. Closed by commit R115:aeac3023a719: Applet: Add configure button next to tabbar (authored by drosca). REPOSITORY R115 Plasma Audio Volume Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4675?vs=11519&id=1

[Differential] [Commented On] D4689: IconItem: Add roundToIconSize property

2017-02-21 Thread David Rosca
drosca added a comment. Yes, I want to use it in plasma-pa applet for volume indicator icons (the small icon next to slider). Currently, they are too small but next round icon size is already too big. This change will make it possible to make them just few pixels bigger. REPOSITORY R242 P

[Differential] [Commented On] D4674: StreamRestore: Cache values in writeChanges until receiving change from pa

2017-02-21 Thread David Rosca
drosca added a comment. That would work too, but then we won't know which properties were actually changed in `StreamRestore::update` and would need to emit changed signals for all properties. It would make the code more clean, but we would lost this optimization (even though is probabl

[Differential] [Accepted] D4674: StreamRestore: Cache values in writeChanges until receiving change from pa

2017-02-21 Thread David Edmundson
davidedmundson accepted this revision. davidedmundson added a comment. This revision is now accepted and ready to land. Ok, I understand the problem. Solution works. Reading this I have an alternative proposal that might make the code simpler: Instead of building pa_ext_stream_restore

[Differential] [Request, 1 line] D4701: [Greeter] Also clear selection clipboard

2017-02-21 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY We only cleared the Ctrl+V clipboard but not middle click paste. TEST PLAN I can no longer middle click paste stuff I selected earlier. REP

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

2017-02-21 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/630/ Project: PLATFORM=Linux,compiler=gcc Date of build: Tue, 21 Feb 2017 09:45:28 + Build duration: 15 min CHANGE SET Revision 8736650c88ee864fd83a3dc544834dc7d0a4