[Breeze] [Bug 374311] it is too easy to activate context menu item if mouse moves during right-click

[Raghavendra kamath]

https://bugs.kde.org/show_bug.cgi?id=374311

--- Comment #7 from Raghavendra kamath  ---
This issue is more serious when you use a pen tablet with programs such as
Krita and Kdenlive. Often times due to pen being not steady the first menu
entry is clicked. This sometimes happens so fast that you can't see the menu
appear fully.

-- 
You are receiving this mail because:
You are the assignee for the bug.

D29619: Improve StatusNotifierTest

[David Redondo]

davidre created this revision.
davidre added a reviewer: kmaterka.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidre requested review of this revision.

REVISION SUMMARY
  Allows to set Icon, AttentionIcon and OverlayIcon either by name or pixmap.
  Makes it easier to test combinations of those.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  snitest (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29619

AFFECTED FILES
  applets/systemtray/tests/statusnotifier/statusnotifiertest.cpp
  applets/systemtray/tests/statusnotifier/statusnotifiertest.ui

To: davidre, kmaterka
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29619: Improve StatusNotifierTest

[David Redondo]

davidre added a comment.


  F8308915: Screenshot_20200511_093148.png 


REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29619

To: davidre, kmaterka
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D28208: Move sni icon handling logic from data engine to applet

[David Redondo]

davidre updated this revision to Diff 82492.
davidre added a comment.


  - Rebase without test

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28208?vs=78982&id=82492

BRANCH
  sni (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D28208

AFFECTED FILES
  applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml
  applets/systemtray/systemtraymodel.cpp
  applets/systemtray/systemtraymodel.h
  dataengines/statusnotifieritem/statusnotifieritemsource.cpp

To: davidre, kmaterka, broulik, mart, #plasma, #vdg, #frameworks
Cc: bruns, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29594: Close low battery notificaton when showing critical

[Kai Uwe Broulik]

This revision was automatically updated to reflect the committed changes.
Closed by commit R122:afff6ae64f42: Close low battery notificaton when showing 
critical (authored by broulik).

REPOSITORY
  R122 Powerdevil

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29594?vs=82422&id=82493

REVISION DETAIL
  https://phabricator.kde.org/D29594

AFFECTED FILES
  daemon/powerdevilcore.cpp

To: broulik, #plasma, nicolasfella
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29475: [Global Shortcuts KCM] Also check PortableText

[Kai Uwe Broulik]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:c6a8b2f5e940: [Global Shortcuts KCM] Also check 
PortableText (authored by broulik).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29475?vs=82073&id=82494

REVISION DETAIL
  https://phabricator.kde.org/D29475

AFFECTED FILES
  kcms/keys/filteredmodel.cpp

To: broulik, #plasma, #vdg, davidre
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29064: Support module commandline paramenter

[Marco Martin]

This revision was automatically updated to reflect the committed changes.
Closed by commit R124:c3debf5fe0f2: Support module commandline paramenter 
(authored by mart).

REPOSITORY
  R124 System Settings

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29064?vs=81319&id=82497

REVISION DETAIL
  https://phabricator.kde.org/D29064

AFFECTED FILES
  app/SettingsBase.cpp
  app/SettingsBase.h
  app/SystemSettingsApp.cpp
  app/main.cpp
  core/BaseMode.cpp
  core/BaseMode.h
  core/MenuItem.cpp
  core/MenuItem.h
  core/ModuleView.cpp
  core/ModuleView.h
  icons/IconMode.cpp
  icons/IconMode.h
  sidebar/SidebarMode.cpp
  sidebar/SidebarMode.h

To: mart, #plasma, ngraham
Cc: broulik, bport, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29157: force systemsettings for kcms

[Marco Martin]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:e67e8d6d92a1: force systemsettings for kcms (authored by 
mart).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29157?vs=82283&id=82498

REVISION DETAIL
  https://phabricator.kde.org/D29157

AFFECTED FILES
  runners/services/servicerunner.cpp

To: mart, #plasma, ngraham
Cc: broulik, ngraham, davidre, plasma-devel, Orage, LeGast00n, 
The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra, mart

D29571: [Kickoff] Add emacs/bash style keybindings

[Mariusz Libera]

mlibera abandoned this revision.
mlibera added a comment.


  > In terms of actions this means:
  > 
  > - exposing KStandardAction::rotateUp/rotateDown somewhere generic
  > - porting from KeyEvent to Shortcuts inside the QML
  
  Thanks for the explanation. This sounds like a time-consuming project for 
someone unfamiliar with that stuff. I'll stick with a locally patched Kickoff 
for now.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29571

To: mlibera, davidedmundson
Cc: davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29621: [kstart] Port to ApplicationLauncherJob and CommandLauncherJob

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, dfaure.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  The existing `--service` is kept for compatibility as it prints out the 
actually registered DBus name.
  A new argument `--application` is added which also takes a desktop //name//. 
It prints the launched PID.

TEST PLAN
  - `kstart5 kwrite` launches kwrite
  - `kstart5 --application org.kde.kwrite` launches kwrites and has it properly 
scoped
  - `kstart5 --application org.kde.kwrite --maximize` launches kwrite and has 
it properly scoped and its window maximized
  - `kstart5 --application org.kde.kwrite --url file:///home/foo/bar` launches 
kwrite with that file open

REPOSITORY
  R126 KDE CLI Utilities

REVISION DETAIL
  https://phabricator.kde.org/D29621

AFFECTED FILES
  kstart/CMakeLists.txt
  kstart/kstart.cpp

To: broulik, #plasma, dfaure
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29544: [applets/systemtray] Show base icon when AttentionIcon not set

[Konrad Materka]

kmaterka added a comment.


  Firstly, AttentionIcon support was introduced in D24865 
. It had a bug, fixed for 5.18 in D29386 
. When I merged branch 5.18 (with D29386 
) into master, by mistake I forgot about 
AttentionIcon handling... This change fixes the incomplete merge.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29544

To: kmaterka, #plasma_workspaces, #plasma, ngraham, broulik, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

5.19 release announce notes

[Jonathan Riddell]

Beta on Thursday so please add features for the announce pronto.

https://share.kde.org/apps/files/?dir=/Community%20Notes/Plasma&fileid=1718757
(sort by modified date, edit Plasma 5.19.md)

or reply here

Jonathan

D29529: Use INTERFACE IMPORTED target type instead of ALIAS for compatibility targets

[Arjen Hiemstra]

This revision was automatically updated to reflect the committed changes.
Closed by commit R111:fc35d85131ed: Use INTERFACE IMPORTED target type instead 
of ALIAS for compatibility targets (authored by ahiemstra).

REPOSITORY
  R111 KSysguard Library

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29529?vs=82263&id=82503

REVISION DETAIL
  https://phabricator.kde.org/D29529

AFFECTED FILES
  KF5SysGuardConfig.cmake.in

To: ahiemstra, #plasma, ngraham, kmaterka
Cc: kmaterka, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29623: Fix Warning about KF5::KSysGuard target being deprecated

[Méven Car]

meven created this revision.
meven added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
meven requested review of this revision.

REVISION SUMMARY
  Warnings such as:
  
CMake Warning at 
/usr/lib/x86_64-linux-gnu/cmake/KF5SysGuard/KF5SysGuardConfig.cmake:37 (me
ssage):
  The KF5:: namespace for KSysGuard targets is deprecated.  Please use
   KSysGuard as namespace.
 Call Stack (most recent call first):
   /usr/share/ECM/find-modules/FindKF5.cmake:74 (find_package)
   CMakeLists.txt:55 (find_package)

TEST PLAN
  cmake does not output those warnings

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29623

AFFECTED FILES
  CMakeLists.txt

To: meven, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29623: Fix Warning about KF5::KSysGuard target being deprecated

[David Edmundson]

davidedmundson added a comment.


  thanks

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29623

To: meven, #plasma, ahiemstra, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29624: Fix Warning about KF5::KSysGuard target being deprecated

[Méven Car]

meven created this revision.
meven added reviewers: Plasma, davidedmundson.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
meven requested review of this revision.

REVISION SUMMARY
  Warnings such as:
  
CMake Warning at 
/usr/lib/x86_64-linux-gnu/cmake/KF5SysGuard/KF5SysGuardConfig.cmake:37 (me
ssage):
  The KF5:: namespace for KSysGuard targets is deprecated.  Please use
   KSysGuard as namespace.
 Call Stack (most recent call first):
   /usr/share/ECM/find-modules/FindKF5.cmake:74 (find_package)
   CMakeLists.txt:55 (find_package)

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29624

AFFECTED FILES
  CMakeLists.txt
  applets/CMakeLists.txt

To: meven, #plasma, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29623: Fix Warning about KF5::KSysGuard target being deprecated

[Méven Car]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:d2c57378479a: Fix Warning about KF5::KSysGuard target 
being deprecated (authored by meven).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29623?vs=82504&id=82506

REVISION DETAIL
  https://phabricator.kde.org/D29623

AFFECTED FILES
  CMakeLists.txt

To: meven, #plasma, ahiemstra, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

KDE CI: Plasma » plasma-workspace » kf5-qt5 FreeBSDQt5.14 - Build # 101 - Failure!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-workspace/job/kf5-qt5%20FreeBSDQt5.14/101/
 Project:
kf5-qt5 FreeBSDQt5.14
 Date of build:
Mon, 11 May 2020 10:58:55 +
 Build duration:
1 min 17 sec and counting
   CONSOLE OUTPUT
  [...truncated 528 lines...][2020-05-11T11:00:08.878Z]  * KF5Baloo, File Searching[2020-05-11T11:00:08.878Z]Needed for the File Search runner.[2020-05-11T11:00:08.878Z] [2020-05-11T11:00:08.878Z] -- The following REQUIRED packages have been found:[2020-05-11T11:00:08.878Z] [2020-05-11T11:00:08.878Z]  * Qt5Qml (required version >= 5.14.2)[2020-05-11T11:00:08.878Z]  * Qt5QmlModels (required version >= 5.14.2)[2020-05-11T11:00:08.878Z]  * Qt5QuickWidgets[2020-05-11T11:00:08.878Z]  * Qt5Test[2020-05-11T11:00:08.878Z]  * KF5Runner (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5NotifyConfig (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5Su (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5NewStuff (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5Wallet (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5IdleTime (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * Qt5Quick (required version >= 5.12.0)[2020-05-11T11:00:08.878Z]  * KF5Declarative (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5KCMUtils (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5DocTools (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5KDELibs4Support (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5Crash (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5GlobalAccel (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5DBusAddons (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5Wayland (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5People (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5ActivitiesStats (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KDED[2020-05-11T11:00:08.878Z]  * KF5Plasma (required version >= 5.70.0)[2020-05-11T11:00:08.878Z]  * KF5PlasmaQuick[2020-05-11T11:00:08.878Z]  * ECM (required version >= 1.6.0)[2020-05-11T11:00:08.878Z]  * KF5Package[2020-05-11T11:00:08.878Z]  * KF5[2020-05-11T11:00:08.878Z]  * Qt5Network (required version >= 5.12.0)[2020-05-11T11:00:08.878Z]  * Qt5Concurrent (required version >= 5.12.0)[2020-05-11T11:00:08.878Z]  * Gettext[2020-05-11T11:00:08.879Z]  * KF5I18n (required version >= 5.70.0)[2020-05-11T11:00:08.879Z]  * KF5TextWidgets (required version >= 5.70.0)[2020-05-11T11:00:08.879Z]  * KF5CoreAddons (required version >= 5.70.0)[2020-05-11T11:00:08.879Z]  * KWinDBusInterface[2020-05-11T11:00:08.879Z]  * KF5Screen[2020-05-11T11:00:08.879Z]  * KScreenLocker (required version >= 5.13.80)[2020-05-11T11:00:08.879Z]  * ScreenSaverDBusInterface[2020-05-11T11:00:08.879Z]  * ZLIB, Support for gzip compressed files and data streams, [2020-05-11T11:00:08.879Z]  * Qt5X11Extras[2020-05-11T11:00:08.879Z]  * Qt5Gui (required version >= 5.12.0)[2020-05-11T11:00:08.879Z]  * Qt5Sql[2020-05-11T11:00:08.879Z]  * Qt5[2020-05-11T11:00:08.879Z]  * Qt5Core[2020-05-11T11:00:08.879Z]  * Qt5Widgets[2020-05-11T11:00:08.879Z]  * Phonon4Qt5 (required version >= 4.6.60), Qt-based audio library[2020-05-11T11:00:08.879Z]  * XCB, X protocol C-language Binding, [2020-05-11T11:00:08.879Z]Required for enabling special X11 multihead mode[2020-05-11T11:00:08.879Z] [2020-05-11T11:00:08.879Z] -- The following RUNTIME packages have not been found:[2020-05-11T11:00:08.879Z] [2020-05-11T11:00:08.879Z]  * AppMenuGtkModule, Application Menu GTK+ Module, [2020-05-11T11:00:08.879Z] [2020-05-11T11:00:08.879Z] -- The following OPTIONAL packages have not been found:[2020-05-11T11:00:08.879Z] [2020-05-11T11:00:08.879Z]  * KF5NetworkManagerQt (required version >= 5.70.0), Qt wrapper for NetworkManager API[2020-05-11T11:00:08.879Z]Needed by geolocation data engine.[2020-05-11T11:00:08.879Z]  * AppStreamQt (required version >= 0.10.6), Access metadata for listing available software, [2020-05-11T11:00:08.879Z]  * Qalculate, Qalculate Library, [2020-05-11T11:00:08.879Z]Needed to enable advanced features of the calculator runner[2020-05-11T11:00:08.879Z] [2020-05-11T11:00:09.137Z] -- Configuring done[2020-05-11T11:00:09.705Z] CMake Error at systemmonitor/CMakeLists.txt:22 (add_executable):[2020-05-11T11:00:09.705Z]   Target "systemmonitor" links to target "KF5::ProcessUi" but the target was[2020-05-11T11:00:09.705Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T11:00:09.705Z]   target, or an ALIAS target is missing?[2020-05-11T11:00:09.705Z] [2020-05-11T11:00:09.705Z] [2020-05-11T11:00:09.705Z] CMake Error at runners/kill/CMakeLists.txt:22 (add_library):[2020-05-11T11:00:09.705Z]   Target "krunner_kill" links to target "KF5::ProcessCore" but the target was[2020-05-11T11:00:09.705Z]   not found.  Perhaps a find_package() call is missing

KDE CI: Plasma » plasma-workspace » kf5-qt5 SUSEQt5.14 - Build # 88 - Failure!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-workspace/job/kf5-qt5%20SUSEQt5.14/88/
 Project:
kf5-qt5 SUSEQt5.14
 Date of build:
Mon, 11 May 2020 10:58:55 +
 Build duration:
1 min 25 sec and counting
   CONSOLE OUTPUT
  [...truncated 521 lines...][2020-05-11T11:00:18.124Z]Needed for the File Search runner.[2020-05-11T11:00:18.124Z] [2020-05-11T11:00:18.124Z] -- The following REQUIRED packages have been found:[2020-05-11T11:00:18.124Z] [2020-05-11T11:00:18.124Z]  * Qt5Qml (required version >= 5.14.1)[2020-05-11T11:00:18.124Z]  * Qt5QmlModels (required version >= 5.14.1)[2020-05-11T11:00:18.124Z]  * Qt5QuickWidgets[2020-05-11T11:00:18.124Z]  * Qt5Test[2020-05-11T11:00:18.124Z]  * KF5Runner (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5NotifyConfig (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5Su (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5NewStuff (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5Wallet (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5IdleTime (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * Qt5Quick (required version >= 5.12.0)[2020-05-11T11:00:18.124Z]  * KF5Declarative (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5KCMUtils (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5DocTools (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5KDELibs4Support (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5Crash (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5GlobalAccel (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5DBusAddons (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5Wayland (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5People (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5ActivitiesStats (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KDED[2020-05-11T11:00:18.124Z]  * KF5Plasma (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5PlasmaQuick[2020-05-11T11:00:18.124Z]  * ECM (required version >= 1.6.0)[2020-05-11T11:00:18.124Z]  * KF5Package[2020-05-11T11:00:18.124Z]  * KF5[2020-05-11T11:00:18.124Z]  * Qt5Network (required version >= 5.12.0)[2020-05-11T11:00:18.124Z]  * Qt5Concurrent (required version >= 5.12.0)[2020-05-11T11:00:18.124Z]  * Gettext[2020-05-11T11:00:18.124Z]  * KF5I18n (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5TextWidgets (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KF5CoreAddons (required version >= 5.70.0)[2020-05-11T11:00:18.124Z]  * KWinDBusInterface[2020-05-11T11:00:18.124Z]  * KF5Screen[2020-05-11T11:00:18.124Z]  * KScreenLocker (required version >= 5.13.80)[2020-05-11T11:00:18.124Z]  * ScreenSaverDBusInterface[2020-05-11T11:00:18.124Z]  * ZLIB, Support for gzip compressed files and data streams, [2020-05-11T11:00:18.124Z]  * Qt5X11Extras[2020-05-11T11:00:18.124Z]  * Qt5Gui (required version >= 5.12.0)[2020-05-11T11:00:18.124Z]  * Qt5Sql[2020-05-11T11:00:18.124Z]  * Qt5[2020-05-11T11:00:18.124Z]  * Qt5Core[2020-05-11T11:00:18.124Z]  * Qt5Widgets[2020-05-11T11:00:18.124Z]  * Phonon4Qt5 (required version >= 4.6.60), Qt-based audio library[2020-05-11T11:00:18.124Z]  * XCB, X protocol C-language Binding, [2020-05-11T11:00:18.124Z]Required for enabling special X11 multihead mode[2020-05-11T11:00:18.124Z] [2020-05-11T11:00:18.124Z] -- The following RUNTIME packages have not been found:[2020-05-11T11:00:18.124Z] [2020-05-11T11:00:18.124Z]  * org.kde.plasma.core-QMLModule, QML module 'org.kde.plasma.core' is a runtime dependency.[2020-05-11T11:00:18.124Z]  * IsoCodes, ISO language, territory, currency, script codes and their translations, [2020-05-11T11:00:18.124Z]Translation of country names in digital clock applet[2020-05-11T11:00:18.124Z]  * AppMenuGtkModule, Application Menu GTK+ Module, [2020-05-11T11:00:18.124Z] [2020-05-11T11:00:18.124Z] -- The following OPTIONAL packages have not been found:[2020-05-11T11:00:18.124Z] [2020-05-11T11:00:18.124Z]  * Qalculate, Qalculate Library, [2020-05-11T11:00:18.124Z]Needed to enable advanced features of the calculator runner[2020-05-11T11:00:18.124Z]  * libgps, GPS support for geolocation, [2020-05-11T11:00:18.124Z] [2020-05-11T11:00:18.124Z] -- Configuring done[2020-05-11T11:00:18.711Z] CMake Error at systemmonitor/CMakeLists.txt:22 (add_executable):[2020-05-11T11:00:18.711Z]   Target "systemmonitor" links to target "KF5::ProcessUi" but the target was[2020-05-11T11:00:18.711Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T11:00:18.711Z]   target, or an ALIAS target is missing?[2020-05-11T11:00:18.711Z] [2020-05-11T11:00:18.711Z] [2020-05-11T11:00:18.711Z] CMake Error at runners/kill/CMakeLists.txt:22 (add_library):[2020-05-11T11:00:18.711Z]   Target "krunner_kill" links to target "KF5::ProcessCore" but the target was[2020-05-11T11:00:18.711Z]   not found.  Perhaps a find_pack

Monday notes meeting for 11/5/2020

[Marco Martin]

Nico:
* Got rid of some deprecated stuff in KDE Connect.
* Discovered that Qt Network Bearer Management is deprecated in 5.15
without clear replacement. Needs discussion:
https://phabricator.kde.org/T13134
Task 13134 "Port away from Qt Network Bearer Management"
[Open,Needs Triage] {KF6} https://phabricator.kde.org/T13134
* Use generated DBus interface in KNotifications
https://phabricator.kde.org/D29420
Diff 29420 "Generate DBus interface" [Needs Review]
https://phabricator.kde.org/D29420
* Made a PoC for KNotifications without notifyrc files, which we want
to kill for several reasons https://phabricator.kde.org/D29405
Diff 29405 "[PoC] Make notifications work without a notifyrc file"
[Needs Review] https://phabricator.kde.org/D29405
* Cleanup and refactoring of the Android build tooling, like removing
Android build specs for Kate[sic!].
* Made a PoC for a Gitlab Android CI based on our existing tooling.
* Next I want to start a discussion on defining a release process for
Android apps since Itinerary and KTrip are close to release maturity
and current "process" is a mess.
* Fixed a Qt bug in Flickable that is affecting Discover and
potentially all apps using CardsListView.
https://codereview.qt-project.org/c/qt/qtdeclarative/+/299787

Kai Uwe:
Notifications:
* Battery *low* notification is now also critical
plasma-browser-integration:
* Now uses HTML5 video "poster" property for album art, when set and
no Media Session in use
Other:
* Global shortcuts KCM search now also finds portable text
* So you can search for e.g. Ctrl without having to know the localized term
* Port Klipper plasmoid to Prison QML import, please review D29478
Diff 29478 "[Clipboard Plasmoid] Port to Prison QML import" [Needs
Review] https://phabricator.kde.org/D29478
Diff 29478 "[Clipboard Plasmoid] Port to Prison QML import" [Needs
Review] https://phabricator.kde.org/D29478

Jonathan:
* so Plasma 5.19 beta is on Thursday!
* please work on the release announcement/new features

David R:
* I had to submit my master's thesis on friday so not much from me but
because of that I was not here last week to anonounce the totally
rewritten super awesome new global shortcuts kcm, now merged :)

Arjen:
* big thing is that we landed the ksysguard work
* so now there's a new daemon, a new library and sparkly new plasmoids
* there's a new ksysguard ui in the works as well, but that won't be
ready for 5.19
* if you're interested, it's now available at
https://invent.kde.org/ahiemstra/ksysguardqml

David E:
Things I've done:
 - worked on some wayland stuff
 - fixed a kcrash in the kscreen KCM
* worryingly we still have quite a few bugs marked as major coming up
to the release of the Plasma beta
*it kinda seems the tagging system isn't helping much (and tbh, I'm
not exactly doing a huge amount either)
*tagged as "VHI" not major
*I would like it if anyone can reproduce
https://bugs.kde.org/show_bug.cgi?id=419008 to let me know
KDE bug 419008 in frameworks-kirigami (general) "Always crash in
Regional Setting within System Settings" [critical,]
https://bugs.kde.org/show_bug.cgi?id=419008

Bhushan:
* I am mostly working for UI side of plasma mobile notifications on lockscreen
* And this weekend is gitlab migration
* apart from it not much plasma related stuff

Marco:
* Last week decicated to the systemmonitor plasmoids and face library:
it landed today:)
* fixed up kcm runner launching modules in systemsettings (and info
modules in kinfocenter mode, they also appear on pwn result category
now) landed today as well

D28487: Redesign of system monitor plasmoids

[Marco Martin]

This revision was not accepted when it landed; it landed in state "Needs 
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:c8deafbb0731: Redesign of system monitor plasmoids 
(authored by mart).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28487?vs=82287&id=82507

REVISION DETAIL
  https://phabricator.kde.org/D28487

AFFECTED FILES
  applets/systemmonitor/CMakeLists.txt
  applets/systemmonitor/common/contents/config/main.xml
  applets/systemmonitor/common/contents/ui/Applet.qml
  applets/systemmonitor/common/contents/ui/ConfigGeneral.qml
  applets/systemmonitor/common/contents/ui/DoublePlotter.qml
  applets/systemmonitor/common/contents/ui/SinglePlotter.qml
  applets/systemmonitor/coreusage/contents/config/faceproperties
  applets/systemmonitor/coreusage/metadata.desktop
  applets/systemmonitor/cpu/Messages.sh
  applets/systemmonitor/cpu/contents/config/config.qml
  applets/systemmonitor/cpu/contents/config/faceproperties
  applets/systemmonitor/cpu/contents/ui/cpu.qml
  applets/systemmonitor/cpu/contents/ui/cpuConfig.qml
  applets/systemmonitor/cpu/metadata.desktop
  applets/systemmonitor/diskactivity/Messages.sh
  applets/systemmonitor/diskactivity/contents/config/config.qml
  applets/systemmonitor/diskactivity/contents/config/faceproperties
  applets/systemmonitor/diskactivity/contents/ui/diskactivity.qml
  applets/systemmonitor/diskactivity/contents/ui/diskactivityConfig.qml
  applets/systemmonitor/diskactivity/metadata.desktop
  applets/systemmonitor/diskusage/Messages.sh
  applets/systemmonitor/diskusage/contents/config/config.qml
  applets/systemmonitor/diskusage/contents/config/faceproperties
  applets/systemmonitor/diskusage/contents/ui/diskusage.qml
  applets/systemmonitor/diskusage/contents/ui/diskusageConfig.qml
  applets/systemmonitor/diskusage/metadata.desktop
  applets/systemmonitor/memory/Messages.sh
  applets/systemmonitor/memory/contents/config/config.qml
  applets/systemmonitor/memory/contents/config/faceproperties
  applets/systemmonitor/memory/contents/ui/memory.qml
  applets/systemmonitor/memory/contents/ui/memoryConfig.qml
  applets/systemmonitor/memory/metadata.desktop
  applets/systemmonitor/net/Messages.sh
  applets/systemmonitor/net/contents/config/config.qml
  applets/systemmonitor/net/contents/config/faceproperties
  applets/systemmonitor/net/contents/ui/displayConfig.qml
  applets/systemmonitor/net/contents/ui/net.qml
  applets/systemmonitor/net/contents/ui/netConfig.qml
  applets/systemmonitor/net/metadata.desktop
  applets/systemmonitor/systemmonitor/CMakeLists.txt
  applets/systemmonitor/systemmonitor/Messages.sh
  applets/systemmonitor/systemmonitor/package/contents/config/config.qml
  applets/systemmonitor/systemmonitor/package/contents/config/main.xml
  
applets/systemmonitor/systemmonitor/package/contents/ui/CompactRepresentation.qml
  applets/systemmonitor/systemmonitor/package/contents/ui/FullRepresentation.qml
  
applets/systemmonitor/systemmonitor/package/contents/ui/config/ConfigAppearance.qml
  
applets/systemmonitor/systemmonitor/package/contents/ui/config/ConfigSensors.qml
  applets/systemmonitor/systemmonitor/package/contents/ui/config/FaceDetails.qml
  applets/systemmonitor/systemmonitor/package/contents/ui/main.qml
  applets/systemmonitor/systemmonitor/package/metadata.desktop
  applets/systemmonitor/systemmonitor/systemmonitor-faces.knsrc
  applets/systemmonitor/systemmonitor/systemmonitor-presets.knsrc
  applets/systemmonitor/systemmonitor/systemmonitor.cpp
  applets/systemmonitor/systemmonitor/systemmonitor.h

To: mart, #plasma
Cc: alexde, ngraham, davidedmundson, mmustac, ahiemstra, plasma-devel, Orage, 
LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, mart

KDE CI: Plasma » plasma-workspace » kf5-qt5 FreeBSDQt5.14 - Build # 102 - Still Failing!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-workspace/job/kf5-qt5%20FreeBSDQt5.14/102/
 Project:
kf5-qt5 FreeBSDQt5.14
 Date of build:
Mon, 11 May 2020 11:09:31 +
 Build duration:
56 sec and counting
   CONSOLE OUTPUT
  [...truncated 546 lines...][2020-05-11T11:10:23.683Z]  * KF5DocTools (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5KDELibs4Support (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5Crash (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5GlobalAccel (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5DBusAddons (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5Wayland (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5People (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5ActivitiesStats (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KDED[2020-05-11T11:10:23.683Z]  * KF5Plasma (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5PlasmaQuick[2020-05-11T11:10:23.683Z]  * ECM (required version >= 1.6.0)[2020-05-11T11:10:23.683Z]  * KF5Package[2020-05-11T11:10:23.683Z]  * KF5[2020-05-11T11:10:23.683Z]  * Qt5Network (required version >= 5.12.0)[2020-05-11T11:10:23.683Z]  * Qt5Concurrent (required version >= 5.12.0)[2020-05-11T11:10:23.683Z]  * Gettext[2020-05-11T11:10:23.683Z]  * KF5I18n (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5TextWidgets (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KF5CoreAddons (required version >= 5.70.0)[2020-05-11T11:10:23.683Z]  * KWinDBusInterface[2020-05-11T11:10:23.683Z]  * KF5Screen[2020-05-11T11:10:23.683Z]  * KScreenLocker (required version >= 5.13.80)[2020-05-11T11:10:23.683Z]  * ScreenSaverDBusInterface[2020-05-11T11:10:23.683Z]  * ZLIB, Support for gzip compressed files and data streams, [2020-05-11T11:10:23.683Z]  * Qt5X11Extras[2020-05-11T11:10:23.683Z]  * Qt5Gui (required version >= 5.12.0)[2020-05-11T11:10:23.683Z]  * Qt5Sql[2020-05-11T11:10:23.683Z]  * Qt5[2020-05-11T11:10:23.683Z]  * Qt5Core[2020-05-11T11:10:23.683Z]  * Qt5Widgets[2020-05-11T11:10:23.683Z]  * Phonon4Qt5 (required version >= 4.6.60), Qt-based audio library[2020-05-11T11:10:23.683Z]  * XCB, X protocol C-language Binding, [2020-05-11T11:10:23.683Z]Required for enabling special X11 multihead mode[2020-05-11T11:10:23.683Z] [2020-05-11T11:10:23.683Z] -- The following RUNTIME packages have not been found:[2020-05-11T11:10:23.683Z] [2020-05-11T11:10:23.683Z]  * AppMenuGtkModule, Application Menu GTK+ Module, [2020-05-11T11:10:23.683Z] [2020-05-11T11:10:23.683Z] -- The following OPTIONAL packages have not been found:[2020-05-11T11:10:23.683Z] [2020-05-11T11:10:23.683Z]  * KF5NetworkManagerQt (required version >= 5.70.0), Qt wrapper for NetworkManager API[2020-05-11T11:10:23.683Z]Needed by geolocation data engine.[2020-05-11T11:10:23.683Z]  * AppStreamQt (required version >= 0.10.6), Access metadata for listing available software, [2020-05-11T11:10:23.683Z]  * Qalculate, Qalculate Library, [2020-05-11T11:10:23.683Z]Needed to enable advanced features of the calculator runner[2020-05-11T11:10:23.683Z] [2020-05-11T11:10:23.941Z] -- Configuring done[2020-05-11T11:10:24.509Z] CMake Error at systemmonitor/CMakeLists.txt:22 (add_executable):[2020-05-11T11:10:24.509Z]   Target "systemmonitor" links to target "KF5::ProcessUi" but the target was[2020-05-11T11:10:24.509Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T11:10:24.509Z]   target, or an ALIAS target is missing?[2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] CMake Error at runners/kill/CMakeLists.txt:22 (add_library):[2020-05-11T11:10:24.509Z]   Target "krunner_kill" links to target "KF5::ProcessCore" but the target was[2020-05-11T11:10:24.509Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T11:10:24.509Z]   target, or an ALIAS target is missing?[2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T11:10:24.509Z]   Target "plasma_applet_systemmonitor" links to target "KF5::SysGuard" but[2020-05-11T11:10:24.509Z]   the target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T11:10:24.509Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T11:10:24.509Z]   Target "plasma_applet_systemmonitor" links to target "KF5::Sensors" but the[2020-05-11T11:10:24.509Z]   target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T11:10:24.509Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] [2020-05-11T11:10:24.509Z] CMake Error at applets/systemmonitor/systemmonito

KDE CI: Plasma » plasma-workspace » kf5-qt5 SUSEQt5.14 - Build # 89 - Still Failing!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-workspace/job/kf5-qt5%20SUSEQt5.14/89/
 Project:
kf5-qt5 SUSEQt5.14
 Date of build:
Mon, 11 May 2020 11:09:31 +
 Build duration:
2 min 7 sec and counting
   CONSOLE OUTPUT
  [...truncated 539 lines...][2020-05-11T11:11:34.231Z]  * KF5KDELibs4Support (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5Crash (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5GlobalAccel (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5DBusAddons (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5Wayland (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5People (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5ActivitiesStats (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KDED[2020-05-11T11:11:34.231Z]  * KF5Plasma (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5PlasmaQuick[2020-05-11T11:11:34.231Z]  * ECM (required version >= 1.6.0)[2020-05-11T11:11:34.231Z]  * KF5Package[2020-05-11T11:11:34.231Z]  * KF5[2020-05-11T11:11:34.231Z]  * Qt5Network (required version >= 5.12.0)[2020-05-11T11:11:34.231Z]  * Qt5Concurrent (required version >= 5.12.0)[2020-05-11T11:11:34.231Z]  * Gettext[2020-05-11T11:11:34.231Z]  * KF5I18n (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5TextWidgets (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KF5CoreAddons (required version >= 5.70.0)[2020-05-11T11:11:34.231Z]  * KWinDBusInterface[2020-05-11T11:11:34.231Z]  * KF5Screen[2020-05-11T11:11:34.231Z]  * KScreenLocker (required version >= 5.13.80)[2020-05-11T11:11:34.231Z]  * ScreenSaverDBusInterface[2020-05-11T11:11:34.231Z]  * ZLIB, Support for gzip compressed files and data streams, [2020-05-11T11:11:34.231Z]  * Qt5X11Extras[2020-05-11T11:11:34.231Z]  * Qt5Gui (required version >= 5.12.0)[2020-05-11T11:11:34.231Z]  * Qt5Sql[2020-05-11T11:11:34.231Z]  * Qt5[2020-05-11T11:11:34.231Z]  * Qt5Core[2020-05-11T11:11:34.231Z]  * Qt5Widgets[2020-05-11T11:11:34.231Z]  * Phonon4Qt5 (required version >= 4.6.60), Qt-based audio library[2020-05-11T11:11:34.231Z]  * XCB, X protocol C-language Binding, [2020-05-11T11:11:34.231Z]Required for enabling special X11 multihead mode[2020-05-11T11:11:34.231Z] [2020-05-11T11:11:34.231Z] -- The following RUNTIME packages have not been found:[2020-05-11T11:11:34.231Z] [2020-05-11T11:11:34.231Z]  * org.kde.plasma.core-QMLModule, QML module 'org.kde.plasma.core' is a runtime dependency.[2020-05-11T11:11:34.231Z]  * IsoCodes, ISO language, territory, currency, script codes and their translations, [2020-05-11T11:11:34.231Z]Translation of country names in digital clock applet[2020-05-11T11:11:34.231Z]  * AppMenuGtkModule, Application Menu GTK+ Module, [2020-05-11T11:11:34.231Z] [2020-05-11T11:11:34.231Z] -- The following OPTIONAL packages have not been found:[2020-05-11T11:11:34.231Z] [2020-05-11T11:11:34.231Z]  * Qalculate, Qalculate Library, [2020-05-11T11:11:34.231Z]Needed to enable advanced features of the calculator runner[2020-05-11T11:11:34.231Z]  * libgps, GPS support for geolocation, [2020-05-11T11:11:34.231Z] [2020-05-11T11:11:35.166Z] -- Configuring done[2020-05-11T11:11:35.730Z] CMake Error at systemmonitor/CMakeLists.txt:22 (add_executable):[2020-05-11T11:11:35.730Z]   Target "systemmonitor" links to target "KF5::ProcessUi" but the target was[2020-05-11T11:11:35.730Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T11:11:35.730Z]   target, or an ALIAS target is missing?[2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] CMake Error at runners/kill/CMakeLists.txt:22 (add_library):[2020-05-11T11:11:35.730Z]   Target "krunner_kill" links to target "KF5::ProcessCore" but the target was[2020-05-11T11:11:35.730Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T11:11:35.730Z]   target, or an ALIAS target is missing?[2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T11:11:35.730Z]   Target "plasma_applet_systemmonitor" links to target "KF5::SysGuard" but[2020-05-11T11:11:35.730Z]   the target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T11:11:35.730Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T11:11:35.730Z]   Target "plasma_applet_systemmonitor" links to target "KF5::Sensors" but the[2020-05-11T11:11:35.730Z]   target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T11:11:35.730Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] [2020-05-11T11:11:35.730Z] CMake Error at applets/systemmonitor/sy

D28922: Port Sensor Face loading from plasmoid

[Marco Martin]

mart closed this revision.

REPOSITORY
  R111 KSysguard Library

REVISION DETAIL
  https://phabricator.kde.org/D28922

To: mart, #plasma, ahiemstra, #vdg
Cc: zzag, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29623: Fix Warning about KF5::KSysGuard target being deprecated

[Arjen Hiemstra]

ahiemstra added a comment.


  Uh, I'm not sure how this fixes anything? It'll remove the warning but it 
means anything using the previous KF5:: namespaced targets is now broken. So 
this at least needs a follow up to change those usages.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29623

To: meven, #plasma, ahiemstra, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29625: Add global shortcut

[Yunhe Guo]

guoyunhe created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
guoyunhe requested review of this revision.

REVISION SUMMARY
  The "Tools" key is function key in some laptops. In ThinkPad + Windows, when 
you press that key, it will open Control Panel or Windows 10 Settings. But in 
Linux, it does nothing. So I think it can be a good shortcut for System 
Settings because no other software use it.

REPOSITORY
  R124 System Settings

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29625

AFFECTED FILES
  app/CMakeLists.txt
  app/systemsettings.khotkeys

To: guoyunhe
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29626: Fix Fix Warning about KF5::ProcessCore target being deprecated

[Méven Car]

meven created this revision.
meven added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
meven requested review of this revision.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29626

AFFECTED FILES
  applets/taskmanager/CMakeLists.txt

To: meven, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29024: feat(wayland): support multiple protocol extensions through plugin system

[Roman Gilg]

romangg added a comment.


  Since this has now the structure with exported headers as requested in 
D29028#653150  and D29028#654526 
 I'll push later today if there are 
no more specific issues pointed out in regards to the code.

REPOSITORY
  R110 KScreen Library

REVISION DETAIL
  https://phabricator.kde.org/D29024

To: romangg, #plasma
Cc: davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29625: Add global shortcut

[Carl Schwan]

ognarb added a comment.


  I think khotkeys is deprecated: https://phabricator.kde.org/T2050

REPOSITORY
  R124 System Settings

REVISION DETAIL
  https://phabricator.kde.org/D29625

To: guoyunhe, #plasma
Cc: ognarb, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29625: Add global shortcut

[Yunhe Guo]

guoyunhe added a comment.


  In D29625#668190 , @ognarb wrote:
  
  > I think khotkeys is deprecated: https://phabricator.kde.org/T2050
  
  
  Not yet, it is delayed to KF6.

REPOSITORY
  R124 System Settings

REVISION DETAIL
  https://phabricator.kde.org/D29625

To: guoyunhe, #plasma
Cc: ognarb, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29628: Use the right targets for KSysGuard

[Arjen Hiemstra]

ahiemstra created this revision.
ahiemstra added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ahiemstra requested review of this revision.

REVISION SUMMARY
  Otherwise the build fails when things try to link to KF5::SysGuard etc.

TEST PLAN
  Builds again

REPOSITORY
  R120 Plasma Workspace

BRANCH
  ksysguard_targets

REVISION DETAIL
  https://phabricator.kde.org/D29628

AFFECTED FILES
  CMakeLists.txt
  dataengines/devicenotifications/CMakeLists.txt
  dataengines/systemmonitor/CMakeLists.txt
  runners/kill/CMakeLists.txt
  systemmonitor/CMakeLists.txt

To: ahiemstra, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29625: Add global shortcut

[David Redondo]

davidre added a comment.


  -1 for khotkeys.

REPOSITORY
  R124 System Settings

REVISION DETAIL
  https://phabricator.kde.org/D29625

To: guoyunhe, #plasma
Cc: davidre, ognarb, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29625: Add global shortcut

[David Redondo]

davidre added a comment.


  In D29625#668203 , @davidre wrote:
  
  > -1 for khotkeys.
  
  
  Add `X-KDE-Shortcuts`  to the desktop file instead and instal it to 
share/kglobalaccel

REPOSITORY
  R124 System Settings

REVISION DETAIL
  https://phabricator.kde.org/D29625

To: guoyunhe, #plasma
Cc: davidre, ognarb, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29544: [applets/systemtray] Show base icon when AttentionIcon not set

[Konrad Materka]

kmaterka added a comment.


  Can we have it reviewed before 5.19 release?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29544

To: kmaterka, #plasma_workspaces, #plasma, ngraham, broulik, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29625: Add global shortcut

[Yunhe Guo]

guoyunhe updated this revision to Diff 82515.
guoyunhe added a comment.


  Use KGlobalAccel instead

REPOSITORY
  R124 System Settings

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29625?vs=82508&id=82515

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29625

AFFECTED FILES
  app/CMakeLists.txt
  app/systemsettings.desktop

To: guoyunhe, #plasma
Cc: davidre, ognarb, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D28662: KCM LookAndFeel check widgetStyle availability before applying it

[Cyril Rossi]

crossi added a comment.


  In D28662#664211 , @heikobecker 
wrote:
  
  > This breaks the lookandfeel-kcm test. I saw this with 5.18.5 but it also 
fails on build.kde.org for a while, e.g.:
  >
  >   14:11:19  FAIL!  : KcmTest::testKCMSave() Compared values are not the same
  >   14:11:19 Actual   (cg.readEntry("widgetStyle", QString())): ""
  >   14:11:19 Expected (QString("testValue"))  : 
"testValue"
  >   14:11:19 Loc: 
[/home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 
SUSEQt5.14/kcms/lookandfeel/autotests/kcmtest.cpp(185)]
  >  
  >
  >
  > 
https://build.kde.org/job/Plasma/job/plasma-desktop/job/stable-kf5-qt5%20SUSEQt5.14/23/console
  
  
  I will have a look at this test case. Thanks for reporting this.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D28662

To: crossi, #plasma, ervin, bport, meven, ngraham
Cc: heikobecker, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29626: Fix Fix Warning about KF5::ProcessCore target being deprecated

[Arjen Hiemstra]

ahiemstra requested changes to this revision.
ahiemstra added a comment.
This revision now requires changes to proceed.


  You need to change `find_package(KF5 REQUIRED COMPONENTS SysGuard)` to 
`find_package(KSysGuard CONFIG REQUIRED)` in the root CMakeLists file.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29626

To: meven, #plasma, ahiemstra
Cc: ahiemstra, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D29628: Use the right targets for KSysGuard

[Méven Car]

meven requested changes to this revision.
meven added a comment.
This revision now requires changes to proceed.


  It miss a KSysGuard::ProcessCore in 
`applets/systemmonitor/systemmonitor/CMakeLists.txt`

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29628

To: ahiemstra, #plasma, meven
Cc: meven, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29628: Use the right targets for KSysGuard

[Méven Car]

meven added a comment.


  ANd `KSysGuard::Sensors` `KSysGuard::SensorFaces` in 
`applets/systemmonitor/systemmonitor/CMakeLists.txt`

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29628

To: ahiemstra, #plasma, meven
Cc: meven, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29628: Use the right targets for KSysGuard

[Arjen Hiemstra]

ahiemstra updated this revision to Diff 82516.
ahiemstra added a comment.


  - Also change system monitor plasmoid

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29628?vs=82513&id=82516

BRANCH
  ksysguard_targets

REVISION DETAIL
  https://phabricator.kde.org/D29628

AFFECTED FILES
  CMakeLists.txt
  applets/systemmonitor/systemmonitor/CMakeLists.txt
  dataengines/devicenotifications/CMakeLists.txt
  dataengines/systemmonitor/CMakeLists.txt
  runners/kill/CMakeLists.txt
  systemmonitor/CMakeLists.txt

To: ahiemstra, #plasma, meven
Cc: meven, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29628: Use the right targets for KSysGuard

[Méven Car]

meven accepted this revision.
meven added a comment.
This revision is now accepted and ready to land.


  LGTM

REPOSITORY
  R120 Plasma Workspace

BRANCH
  ksysguard_targets

REVISION DETAIL
  https://phabricator.kde.org/D29628

To: ahiemstra, #plasma, meven
Cc: meven, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29626: Fix Fix Warning about KF5::ProcessCore target being deprecated

[Méven Car]

meven updated this revision to Diff 82518.
meven added a comment.


  Update root CMakeLists.txt

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29626?vs=82509&id=82518

BRANCH
  arcpatch-D29626

REVISION DETAIL
  https://phabricator.kde.org/D29626

AFFECTED FILES
  CMakeLists.txt
  applets/taskmanager/CMakeLists.txt

To: meven, #plasma, ahiemstra
Cc: ahiemstra, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D29628: Use the right targets for KSysGuard

[Méven Car]

meven added a comment.


  Related fix in `plasma-desktop` https://phabricator.kde.org/D29626

REPOSITORY
  R120 Plasma Workspace

BRANCH
  ksysguard_targets

REVISION DETAIL
  https://phabricator.kde.org/D29628

To: ahiemstra, #plasma, meven
Cc: meven, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D28208: Move sni icon handling logic from data engine to applet

[David Redondo]

davidre updated this revision to Diff 82519.
davidre added a comment.


  - fix

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28208?vs=82492&id=82519

BRANCH
  sni (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D28208

AFFECTED FILES
  applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml
  applets/systemtray/systemtraymodel.cpp
  applets/systemtray/systemtraymodel.h
  dataengines/statusnotifieritem/statusnotifieritemsource.cpp

To: davidre, kmaterka, broulik, mart, #plasma, #vdg, #frameworks
Cc: bruns, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29619: Improve StatusNotifierTest

[David Redondo]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:5459a533974e: Improve StatusNotifierTest (authored by 
davidre).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29619?vs=82491&id=82521

REVISION DETAIL
  https://phabricator.kde.org/D29619

AFFECTED FILES
  applets/systemtray/tests/statusnotifier/statusnotifiertest.cpp
  applets/systemtray/tests/statusnotifier/statusnotifiertest.ui

To: davidre, kmaterka, apol
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

KDE CI: Plasma » plasma-workspace » kf5-qt5 FreeBSDQt5.14 - Build # 103 - Still Failing!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-workspace/job/kf5-qt5%20FreeBSDQt5.14/103/
 Project:
kf5-qt5 FreeBSDQt5.14
 Date of build:
Mon, 11 May 2020 12:45:11 +
 Build duration:
2 min 3 sec and counting
   CONSOLE OUTPUT
  [...truncated 546 lines...][2020-05-11T12:47:10.840Z]  * KF5DocTools (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5KDELibs4Support (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5Crash (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5GlobalAccel (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5DBusAddons (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5Wayland (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5People (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5ActivitiesStats (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KDED[2020-05-11T12:47:10.840Z]  * KF5Plasma (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5PlasmaQuick[2020-05-11T12:47:10.840Z]  * ECM (required version >= 1.6.0)[2020-05-11T12:47:10.840Z]  * KF5Package[2020-05-11T12:47:10.840Z]  * KF5[2020-05-11T12:47:10.840Z]  * Qt5Network (required version >= 5.12.0)[2020-05-11T12:47:10.840Z]  * Qt5Concurrent (required version >= 5.12.0)[2020-05-11T12:47:10.840Z]  * Gettext[2020-05-11T12:47:10.840Z]  * KF5I18n (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5TextWidgets (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KF5CoreAddons (required version >= 5.70.0)[2020-05-11T12:47:10.840Z]  * KWinDBusInterface[2020-05-11T12:47:10.840Z]  * KF5Screen[2020-05-11T12:47:10.840Z]  * KScreenLocker (required version >= 5.13.80)[2020-05-11T12:47:10.840Z]  * ScreenSaverDBusInterface[2020-05-11T12:47:10.840Z]  * ZLIB, Support for gzip compressed files and data streams, [2020-05-11T12:47:10.840Z]  * Qt5X11Extras[2020-05-11T12:47:10.840Z]  * Qt5Gui (required version >= 5.12.0)[2020-05-11T12:47:10.840Z]  * Qt5Sql[2020-05-11T12:47:10.840Z]  * Qt5[2020-05-11T12:47:10.840Z]  * Qt5Core[2020-05-11T12:47:10.840Z]  * Qt5Widgets[2020-05-11T12:47:10.840Z]  * Phonon4Qt5 (required version >= 4.6.60), Qt-based audio library[2020-05-11T12:47:10.840Z]  * XCB, X protocol C-language Binding, [2020-05-11T12:47:10.840Z]Required for enabling special X11 multihead mode[2020-05-11T12:47:10.840Z] [2020-05-11T12:47:10.840Z] -- The following RUNTIME packages have not been found:[2020-05-11T12:47:10.840Z] [2020-05-11T12:47:10.840Z]  * AppMenuGtkModule, Application Menu GTK+ Module, [2020-05-11T12:47:10.840Z] [2020-05-11T12:47:10.840Z] -- The following OPTIONAL packages have not been found:[2020-05-11T12:47:10.840Z] [2020-05-11T12:47:10.840Z]  * KF5NetworkManagerQt (required version >= 5.70.0), Qt wrapper for NetworkManager API[2020-05-11T12:47:10.840Z]Needed by geolocation data engine.[2020-05-11T12:47:10.840Z]  * AppStreamQt (required version >= 0.10.6), Access metadata for listing available software, [2020-05-11T12:47:10.840Z]  * Qalculate, Qalculate Library, [2020-05-11T12:47:10.840Z]Needed to enable advanced features of the calculator runner[2020-05-11T12:47:10.840Z] [2020-05-11T12:47:11.409Z] -- Configuring done[2020-05-11T12:47:11.669Z] CMake Error at systemmonitor/CMakeLists.txt:22 (add_executable):[2020-05-11T12:47:11.669Z]   Target "systemmonitor" links to target "KF5::ProcessUi" but the target was[2020-05-11T12:47:11.669Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T12:47:11.669Z]   target, or an ALIAS target is missing?[2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] CMake Error at runners/kill/CMakeLists.txt:22 (add_library):[2020-05-11T12:47:11.669Z]   Target "krunner_kill" links to target "KF5::ProcessCore" but the target was[2020-05-11T12:47:11.669Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T12:47:11.669Z]   target, or an ALIAS target is missing?[2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T12:47:11.669Z]   Target "plasma_applet_systemmonitor" links to target "KF5::SysGuard" but[2020-05-11T12:47:11.669Z]   the target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T12:47:11.669Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T12:47:11.669Z]   Target "plasma_applet_systemmonitor" links to target "KF5::Sensors" but the[2020-05-11T12:47:11.669Z]   target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T12:47:11.669Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] [2020-05-11T12:47:11.669Z] CMake Error at applets/systemmonitor/systemm

KDE CI: Plasma » plasma-workspace » kf5-qt5 SUSEQt5.14 - Build # 90 - Still Failing!

[CI System]

BUILD FAILURE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-workspace/job/kf5-qt5%20SUSEQt5.14/90/
 Project:
kf5-qt5 SUSEQt5.14
 Date of build:
Mon, 11 May 2020 12:45:11 +
 Build duration:
2 min 10 sec and counting
   CONSOLE OUTPUT
  [...truncated 539 lines...][2020-05-11T12:47:19.594Z]  * KF5KDELibs4Support (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5Crash (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5GlobalAccel (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5DBusAddons (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5Wayland (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5People (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5ActivitiesStats (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KDED[2020-05-11T12:47:19.594Z]  * KF5Plasma (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5PlasmaQuick[2020-05-11T12:47:19.594Z]  * ECM (required version >= 1.6.0)[2020-05-11T12:47:19.594Z]  * KF5Package[2020-05-11T12:47:19.594Z]  * KF5[2020-05-11T12:47:19.594Z]  * Qt5Network (required version >= 5.12.0)[2020-05-11T12:47:19.594Z]  * Qt5Concurrent (required version >= 5.12.0)[2020-05-11T12:47:19.594Z]  * Gettext[2020-05-11T12:47:19.594Z]  * KF5I18n (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5TextWidgets (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KF5CoreAddons (required version >= 5.70.0)[2020-05-11T12:47:19.594Z]  * KWinDBusInterface[2020-05-11T12:47:19.594Z]  * KF5Screen[2020-05-11T12:47:19.594Z]  * KScreenLocker (required version >= 5.13.80)[2020-05-11T12:47:19.594Z]  * ScreenSaverDBusInterface[2020-05-11T12:47:19.594Z]  * ZLIB, Support for gzip compressed files and data streams, [2020-05-11T12:47:19.594Z]  * Qt5X11Extras[2020-05-11T12:47:19.594Z]  * Qt5Gui (required version >= 5.12.0)[2020-05-11T12:47:19.594Z]  * Qt5Sql[2020-05-11T12:47:19.594Z]  * Qt5[2020-05-11T12:47:19.594Z]  * Qt5Core[2020-05-11T12:47:19.594Z]  * Qt5Widgets[2020-05-11T12:47:19.594Z]  * Phonon4Qt5 (required version >= 4.6.60), Qt-based audio library[2020-05-11T12:47:19.594Z]  * XCB, X protocol C-language Binding, [2020-05-11T12:47:19.594Z]Required for enabling special X11 multihead mode[2020-05-11T12:47:19.594Z] [2020-05-11T12:47:19.594Z] -- The following RUNTIME packages have not been found:[2020-05-11T12:47:19.594Z] [2020-05-11T12:47:19.594Z]  * org.kde.plasma.core-QMLModule, QML module 'org.kde.plasma.core' is a runtime dependency.[2020-05-11T12:47:19.594Z]  * IsoCodes, ISO language, territory, currency, script codes and their translations, [2020-05-11T12:47:19.594Z]Translation of country names in digital clock applet[2020-05-11T12:47:19.594Z]  * AppMenuGtkModule, Application Menu GTK+ Module, [2020-05-11T12:47:19.594Z] [2020-05-11T12:47:19.594Z] -- The following OPTIONAL packages have not been found:[2020-05-11T12:47:19.594Z] [2020-05-11T12:47:19.594Z]  * Qalculate, Qalculate Library, [2020-05-11T12:47:19.594Z]Needed to enable advanced features of the calculator runner[2020-05-11T12:47:19.594Z]  * libgps, GPS support for geolocation, [2020-05-11T12:47:19.594Z] [2020-05-11T12:47:19.852Z] -- Configuring done[2020-05-11T12:47:20.109Z] CMake Error at systemmonitor/CMakeLists.txt:22 (add_executable):[2020-05-11T12:47:20.109Z]   Target "systemmonitor" links to target "KF5::ProcessUi" but the target was[2020-05-11T12:47:20.109Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T12:47:20.109Z]   target, or an ALIAS target is missing?[2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] CMake Error at runners/kill/CMakeLists.txt:22 (add_library):[2020-05-11T12:47:20.109Z]   Target "krunner_kill" links to target "KF5::ProcessCore" but the target was[2020-05-11T12:47:20.109Z]   not found.  Perhaps a find_package() call is missing for an IMPORTED[2020-05-11T12:47:20.109Z]   target, or an ALIAS target is missing?[2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T12:47:20.109Z]   Target "plasma_applet_systemmonitor" links to target "KF5::SysGuard" but[2020-05-11T12:47:20.109Z]   the target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T12:47:20.109Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] CMake Error at applets/systemmonitor/systemmonitor/CMakeLists.txt:5 (add_library):[2020-05-11T12:47:20.109Z]   Target "plasma_applet_systemmonitor" links to target "KF5::Sensors" but the[2020-05-11T12:47:20.109Z]   target was not found.  Perhaps a find_package() call is missing for an[2020-05-11T12:47:20.109Z]   IMPORTED target, or an ALIAS target is missing?[2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] [2020-05-11T12:47:20.109Z] CMake Error at applets/systemmonitor/s

D29628: Use the right targets for KSysGuard

[Arjen Hiemstra]

This revision was automatically updated to reflect the committed changes.
Closed by commit R120:8ae9b561c82f: Use the right targets for KSysGuard 
(authored by ahiemstra).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29628?vs=82516&id=82522

REVISION DETAIL
  https://phabricator.kde.org/D29628

AFFECTED FILES
  CMakeLists.txt
  applets/systemmonitor/systemmonitor/CMakeLists.txt
  dataengines/devicenotifications/CMakeLists.txt
  dataengines/systemmonitor/CMakeLists.txt
  runners/kill/CMakeLists.txt
  systemmonitor/CMakeLists.txt

To: ahiemstra, #plasma, meven
Cc: meven, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29630: Use the new KSysGuard namespace for KSysGuard related targets

[Arjen Hiemstra]

ahiemstra created this revision.
ahiemstra added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ahiemstra requested review of this revision.

REVISION SUMMARY
  The targets in the KF5 namespace have been deprecated. This updates the
  cmake files to use the new targets.

TEST PLAN
  Builds correctly with most recent libksysguard.

REPOSITORY
  R845 Plasma Vault

BRANCH
  ksysguard_targets

REVISION DETAIL
  https://phabricator.kde.org/D29630

AFFECTED FILES
  CMakeLists.txt
  kded/CMakeLists.txt

To: ahiemstra, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29570: Fix some alignments

[Niccolò Venerandi]

niccolove added inline comments.

INLINE COMMENTS

> ndavis wrote in PowerManagementItem.qml:34
> How does this affect RTL layouts?

It does look a bit too close on the right border in LTS. I guess I should add a 
rightMargin here?

REPOSITORY
  R120 Plasma Workspace

BRANCH
  fixspacing (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29570

To: niccolove, #vdg, #plasma, ngraham
Cc: ndavis, abetts, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, ahiemstra, mart

KDE CI: Plasma » plasma-workspace » kf5-qt5 FreeBSDQt5.14 - Build # 104 - Still unstable!

[CI System]

BUILD UNSTABLE
 Build URL
https://build.kde.org/job/Plasma/job/plasma-workspace/job/kf5-qt5%20FreeBSDQt5.14/104/
 Project:
kf5-qt5 FreeBSDQt5.14
 Date of build:
Mon, 11 May 2020 12:48:21 +
 Build duration:
7 min 7 sec and counting
   JUnit Tests
  Name: projectroot.kioslave.desktop Failed: 1 test(s), Passed: 0 test(s), Skipped: 0 test(s), Total: 1 test(s)Failed: projectroot.kioslave.desktop.tests.testdesktopName: projectroot.klipper Failed: 0 test(s), Passed: 2 test(s), Skipped: 0 test(s), Total: 2 test(s)Name: projectroot.libcolorcorrect Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.libkworkspace Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.libtaskmanager Failed: 0 test(s), Passed: 2 test(s), Skipped: 0 test(s), Total: 2 test(s)Name: projectroot.runners.bookmarks Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.runners.services Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.shell Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)

D29603: Adds custom spacing between task manager icons

[Niccolò Venerandi]

niccolove added a comment.


  Are we really sure this is worth exposing to the user?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29603

To: martellini, #vdg
Cc: niccolove, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29623: Fix Warning about KF5::KSysGuard target being deprecated

[Méven Car]

meven added a comment.


  Improved by D29628 

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29623

To: meven, #plasma, ahiemstra, davidedmundson
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29626: Fix Warning about KF5::ProcessCore target being deprecated

[Méven Car]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:baa168ede063: Fix Warning about KF5::ProcessCore target 
being deprecated (authored by meven).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29626?vs=82518&id=82524

REVISION DETAIL
  https://phabricator.kde.org/D29626

AFFECTED FILES
  CMakeLists.txt
  applets/taskmanager/CMakeLists.txt

To: meven, #plasma, ahiemstra
Cc: ahiemstra, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D29603: Adds custom spacing between task manager icons

[Nathaniel Graham]

ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.


  I think it would make more sense to fix the default sizing behavior rather 
than adding these. We already have an "icon size" setting which roughly adjusts 
this already, just not very well. That should be fixed rather than adding 
additional user-facing settings, or at least before doing so.
  
  Also, it doesn't make sense IMO to expose separate horizontal and vertical 
controls; there should be just one control visible, and it should affect the 
task manager no matter its orientation.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29603

To: martellini, #vdg, ngraham
Cc: ngraham, niccolove, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29616: Fix memory errors caused by using dangling pointers to SensorClients in SensorAgent

[Jiří Paleček]

jpalecek added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in SensorBrowser.cpp:53
> When you delete the map content, all agents should loose their connections, 
> no?

No, it only deletes the HostInfo structure. See the source 
.
 It doesn't disconnect anything. Nor does deleting `this`, as the Agent holds 
the client as a bare pointer, it can't know that it ceased to exist. It isn't 
even a `QPointer` (probably bc. `SensorClient` isn't a `QObject`).

REPOSITORY
  R106 KSysguard

REVISION DETAIL
  https://phabricator.kde.org/D29616

To: jpalecek, davidedmundson, ahiemstra
Cc: anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29548: Drag and drop assigning windows to activities in the default switcher

[Nathaniel Graham]

ngraham added a comment.


  So cool! Can you make the activity that the window is being dragged into 
highlight or otherwise somehow provide a visual indication that it's going to 
accept the drop?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29548

To: ivan, #plasma, mart, davidedmundson, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27203: Don't try to open files we can't figure out where they are

[Nathaniel Graham]

ngraham added a comment.


  Does this fix https://bugs.kde.org/show_bug.cgi?id=408174?

REPOSITORY
  R135 Integration for Qt applications in Plasma

REVISION DETAIL
  https://phabricator.kde.org/D27203

To: apol, #frameworks, #plasma, dfaure, meven, ahmadsamir
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29024: feat(wayland): support multiple protocol extensions through plugin system

[David Edmundson]

davidedmundson added a comment.


  It's still got this pointless plugin within plugin situation.
  
  > For that at least one Wayland event loop has to be launched
  
  It needs a roundrip after the registry is fetched. That doesn't need an event 
loop.

REPOSITORY
  R110 KScreen Library

REVISION DETAIL
  https://phabricator.kde.org/D29024

To: romangg, #plasma
Cc: davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29632: Use better collator settings

[David Redondo]

davidre created this revision.
davidre added reviewers: ognarb, broulik, Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidre requested review of this revision.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  collator (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29632

AFFECTED FILES
  kcms/keys/shortcutsmodel.cpp

To: davidre, ognarb, broulik, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29633: [Battery Monitor] Fix toolTipMainText when plugged in

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, ngraham.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  It only had a return value for "NoCharge" since the else case was only for 
not plugged in, resulting in it effectively returning `undefined`, throwing 
during assignment "cannot assign [undefined] to string" and leaving whatever 
text happened to be shown last.

TEST PLAN
  Literally none

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29633

AFFECTED FILES
  applets/batterymonitor/package/contents/ui/batterymonitor.qml

To: broulik, #plasma, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D28154: Add users KCM

[Yuri Chornoivan]

yurchor added a subscriber: ltoscano.
yurchor added a comment.


  Please do not get me wrong (I'm fine with the new designs and updating the 
code and this new KCM is very good), but it would be good to hear some words 
from @ltoscano on the catalog (and KCM) naming.
  
  It is also evident that all the new KCMs do not have the "Help" button. Is it 
the new policy?
  
  Thanks in advance for your comments.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  arcpatch-D28154

REVISION DETAIL
  https://phabricator.kde.org/D28154

To: cblack, #plasma, #vdg, ngraham
Cc: ltoscano, mart, yurchor, iasensio, meven, crossi, The-Feren-OS-Dev, 
davidedmundson, broulik, filipf, ngraham, nicolasfella, zzag, plasma-devel, 
Orage, LeGast00n, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra

D27203: Don't try to open files we can't figure out where they are

[Méven Car]

meven accepted this revision.
meven added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> kdeplatformfiledialoghelper.cpp:226
> +//passing non-local files as the working directory is not supported.
> +//See QFileDialogPrivate::initialSelections
> +//Selectingg files should be done through the correct method.

QFileDialogPrivate::initialSelection

> kdeplatformfiledialoghelper.cpp:227
> +//See QFileDialogPrivate::initialSelections
> +//Selectingg files should be done through the correct method.
> +m_fileWidget->setUrl(directory);

typo Selectingg

REPOSITORY
  R135 Integration for Qt applications in Plasma

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D27203

To: apol, #frameworks, #plasma, dfaure, meven, ahmadsamir
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D28154: Add users KCM

[Luigi Toscano]

ltoscano added a comment.


  Technically kcm should be called kcm_. So far the 
Plasma_version part has been ignored, but at least kcm_ should be there.
  
  I'm not sure about the lack of the button.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  arcpatch-D28154

REVISION DETAIL
  https://phabricator.kde.org/D28154

To: cblack, #plasma, #vdg, ngraham
Cc: ltoscano, mart, yurchor, iasensio, meven, crossi, The-Feren-OS-Dev, 
davidedmundson, broulik, filipf, ngraham, nicolasfella, zzag, plasma-devel, 
Orage, LeGast00n, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, 
ahiemstra

D29625: Add global shortcut

[Yunhe Guo]

guoyunhe updated this revision to Diff 82532.
guoyunhe added a comment.


  Create symlink

REPOSITORY
  R124 System Settings

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29625?vs=82515&id=82532

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29625

AFFECTED FILES
  app/CMakeLists.txt
  app/systemsettings.desktop

To: guoyunhe, #plasma
Cc: davidre, ognarb, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D26934: KCM/Autostart Add a model to separate logic from UI

[Méven Car]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:01776488ab8b: KCM/Autostart Add a model to separate logic 
from UI (authored by meven).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D26934?vs=80200&id=82533

REVISION DETAIL
  https://phabricator.kde.org/D26934

AFFECTED FILES
  kcms/autostart/CMakeLists.txt
  kcms/autostart/autostart.cpp
  kcms/autostart/autostart.h
  kcms/autostart/autostartitem.cpp
  kcms/autostart/autostartitem.h
  kcms/autostart/autostartmodel.cpp
  kcms/autostart/autostartmodel.h

To: meven, mlaurent, ervin, #plasma, broulik, bport, crossi
Cc: alex, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29140: Fix POTD in lock screen (partly)

[Yunhe Guo]

guoyunhe added a comment.


  I was looking for kded documentation but cannot find a good example. Do you 
know any?

REPOSITORY
  R114 Plasma Addons

REVISION DETAIL
  https://phabricator.kde.org/D29140

To: guoyunhe, #plasma, ngraham
Cc: davidre, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29024: feat(wayland): support multiple protocol extensions through plugin system

[Roman Gilg]

romangg added a comment.


  These are not specific issues but some general complains about the overall 
concept chosen here without providing an alternative solution.
  
  I chose this plugin system because it allows robust extension to the current 
system. Plugins are well contained and reuse already existing infrastructure. 
Besides moving the code around there are practically no logic changes to the 
KWayland backend.
  
  Note that I need to have this plugin system or something similar in for 5.19 
or I will be forced to fork libkscreen permanently for KWinFT. I would like to 
omit this and instead continue my work on libkscreen as a KDE project like I 
have worked on it in the last two years.

REPOSITORY
  R110 KScreen Library

REVISION DETAIL
  https://phabricator.kde.org/D29024

To: romangg, #plasma
Cc: davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29635: [kcms/keys] Make saving sync

[David Redondo]

davidre created this revision.
davidre added reviewers: broulik, Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidre requested review of this revision.

REVISION SUMMARY
  If it's async it will not save when changing modules, or quitting kcmshell, 
see T13140 .
  BUG: 421333

TEST PLAN
  Make a change, change module in systemsettings. Changes are saved.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  syncSave (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29635

AFFECTED FILES
  kcms/keys/shortcutsmodel.cpp

To: davidre, broulik, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29616: Fix memory errors caused by using dangling pointers to SensorClients in SensorAgent

[Arjen Hiemstra]

ahiemstra requested changes to this revision.
ahiemstra added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> jpalecek wrote in SensorBrowser.cpp:53
> No, it only deletes the HostInfo structure. See the source 
> .
>  It doesn't disconnect anything. Nor does deleting `this`, as the Agent holds 
> the client as a bare pointer, it can't know that it ceased to exist. It isn't 
> even a `QPointer` (probably bc. `SensorClient` isn't a `QObject`).

I agree with Anthony though, if you delete a hostInfo it _should_ remove its 
connection. So it's better to move this code into a destructor for HostInfo.

> ksysguard.cpp:153
> +{
> +  if (KSGRD::SensorMgr) KSGRD::SensorMgr->disconnectClient(this);
> +}

Coding style: Single line ifs still need braces and should not be on a single 
line.

REPOSITORY
  R106 KSysguard

REVISION DETAIL
  https://phabricator.kde.org/D29616

To: jpalecek, davidedmundson, ahiemstra
Cc: anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29616: Fix memory errors caused by using dangling pointers to SensorClients in SensorAgent

[Jiří Paleček]

jpalecek added inline comments.

INLINE COMMENTS

> ahiemstra wrote in SensorBrowser.cpp:53
> I agree with Anthony though, if you delete a hostInfo it _should_ remove its 
> connection. So it's better to move this code into a destructor for HostInfo.

But the host info, at least in its present form, doesn't have the client 
pointer to pass to `disconnectClient`. But I can see your point, I will change 
it.

REPOSITORY
  R106 KSysguard

REVISION DETAIL
  https://phabricator.kde.org/D29616

To: jpalecek, davidedmundson, ahiemstra
Cc: anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29632: [kcms/keys] Use better collator settings

[Aleix Pol Gonzalez]

apol accepted this revision.
apol added a comment.
This revision is now accepted and ready to land.


  It could make sense to leverage QCollatorSortKey in this class if it's 
sorting often.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  collator (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29632

To: davidre, ognarb, broulik, #plasma, apol
Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D29636: [applets/batterymonitor] Fix tooltip main text not updating while plugged in and charging

[Nathaniel Graham]

ngraham created this revision.
ngraham added reviewers: broulik, Plasma, Localization.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  Current code only updates the text on the "not charging" condition, causing 
the tooltip
  display to get out of sync with the rest of the applet once charging begins. 
This patch
  fixes that.
  
  This is a bugfix for a bug that affects Plasma 5.18, but fixing it requires 
introducing a
  new string. As such, I am targeting 5.19, but if the #Localization 
 team approves, I can
  land this on the stable branch if it's accepted.
  
  BUG: 20920
  FIXED-IN: 5.19.0

TEST PLAN
  F8309775: tooltip updates now.png 

REPOSITORY
  R120 Plasma Workspace

BRANCH
  fix-non-updating-battery-tooltip (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29636

AFFECTED FILES
  applets/batterymonitor/package/contents/ui/batterymonitor.qml

To: ngraham, broulik, #plasma, #localization
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29603: Adds custom spacing between task manager icons

[Jacopo Martellini]

martellini added a comment.


  In D29603#668283 , @niccolove 
wrote:
  
  > Are we really sure this is worth exposing to the user?
  
  
  Not sure at all. I just can't stand the default spacing, because it feels 
like the icons are crammed. At least if compared with Win10 or Gnome with dash 
to panel.
  
  In D29603#668297 , @ngraham wrote:
  
  > I think it would make more sense to fix the default sizing behavior rather 
than adding these. We already have an "icon size" setting which roughly adjusts 
this already, just not very well. That should be fixed rather than adding 
additional user-facing settings, or at least before doing so.
  >
  > Also, it doesn't make sense IMO to expose separate horizontal and vertical 
controls; there should be just one control visible, and it should affect the 
task manager no matter its orientation.
  
  
  If that's the better way I totally agree, but I don't know recall that 
setting. Is there an open issue that I can look into?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29603

To: martellini, #vdg, ngraham
Cc: ngraham, niccolove, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29632: [kcms/keys] Use better collator settings

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> shortcutsmodel.cpp:89
>  QCollator collator;
> +collator.setCaseSensitivity(Qt::CaseInsensitive);
>  std::sort(m_components.begin(), m_components.end(), 
> [&](const Component &c1, const Component &c2){

Why no numeric mode here?

REPOSITORY
  R119 Plasma Desktop

BRANCH
  collator (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29632

To: davidre, ognarb, broulik, #plasma, apol
Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D29637: Introduce ImageColors

[Marco Martin]

mart created this revision.
mart added a reviewer: Kirigami.
Herald added a project: Kirigami.
Herald added a subscriber: plasma-devel.
mart requested review of this revision.

REVISION SUMMARY
  ImageColors is a class to extract color statistics out of an image. it 
supports 
  as source :
  
  - Grabbing a static frame out of any Item
  - A QImage or a QIcon (for instance coming from a QAbstractItemModel
  - An icon name from the theme
  
  By a pixel-by pixel analisys it uses a K-means clustering algorythm 
  https://en.wikipedia.org/wiki/K-means_clustering
  that groups averages of colors group that have a computed "distance" less 
than an 
  arbitrary value and so extracts a palette of the "dominant" colors of the 
image,
  having also other imformations such as the % occurrence of each cluster, a 
  complementary color for each cluster, the global average color, the most 
  "vibrant" color of the image, usable as an accent, and good colors for text 
and 
  background, based on the extracted palette

REPOSITORY
  R169 Kirigami

BRANCH
  mart/imageColors

REVISION DETAIL
  https://phabricator.kde.org/D29637

AFFECTED FILES
  src/CMakeLists.txt
  src/colorutils.cpp
  src/colorutils.h
  src/imagecolors.cpp
  src/imagecolors.h
  src/kirigamiplugin.cpp
  tests/imagecolorstest.qml

To: mart, #kirigami
Cc: plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, 
ahiemstra, davidedmundson, mart

D29632: [kcms/keys] Use better collator settings

[David Redondo]

davidre added inline comments.

INLINE COMMENTS

> shortcutsmodel.cpp:89
>  QCollator collator;
> +collator.setCaseSensitivity(Qt::CaseInsensitive);
>  std::sort(m_components.begin(), m_components.end(), 
> [&](const Component &c1, const Component &c2){

We typically don't have apps with numbers but I can also add it?

REPOSITORY
  R119 Plasma Desktop

BRANCH
  collator (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29632

To: davidre, ognarb, broulik, #plasma, apol
Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D29637: Introduce ImageColors

[Carson Black]

cblack added inline comments.

INLINE COMMENTS

> imagecolorstest.qml:1
> +
> +import QtQuick 2.12

There's no test cases here, this should probably go in examples.

REPOSITORY
  R169 Kirigami

REVISION DETAIL
  https://phabricator.kde.org/D29637

To: mart, #kirigami
Cc: cblack, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, 
apol, ahiemstra, davidedmundson, mart

D29636: [applets/batterymonitor] Fix tooltip main text not updating while plugged in and charging

[Luigi Toscano]

ltoscano added a comment.


  Adding a few more details to make it easier to decide on the localization 
side:
  
  - Plasma 5.18 is the new LTS
  - its next releases (5.18.6) is currently planned for September 17th, 2020.
  
  I'd say there is enough time to fix this translation, and having the correct 
information about the charging status of the battery is important enough to 
have the fix in the LTS branch.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29636

To: ngraham, broulik, #plasma, #localization
Cc: ltoscano, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29636: [applets/batterymonitor] Fix tooltip main text not updating while plugged in and charging

[Yuri Chornoivan]

yurchor accepted this revision as: yurchor.
yurchor added a comment.
This revision is now accepted and ready to land.


  Thanks. No objections for porting this into a stable translation branch from 
the Ukrainian team.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  fix-non-updating-battery-tooltip (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29636

To: ngraham, broulik, #plasma, #localization, yurchor
Cc: yurchor, ltoscano, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29644: Select application after it is added

[David Redondo]

davidre created this revision.
davidre added reviewers: broulik, ngraham, Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidre requested review of this revision.

REVISION SUMMARY
  After adding an application, an user will probably want to edit shortcuts for 
  the new application. Having to select it again is cumbersome.

TEST PLAN
  Add an application.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  addIndex (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29644

AFFECTED FILES
  kcms/keys/package/contents/ui/main.qml

To: davidre, broulik, ngraham, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29462: Fix Kirigami.Units.devicePixelRatio=1.3 when it should be 1.0 at 96dpi

[Marco Martin]

mart added a comment.


  the different Units.qml depend from the style, QT_QUICK_CONTROLS_STYLE
  all of them, including those in plasma-framework should be updated at once

REPOSITORY
  R169 Kirigami

REVISION DETAIL
  https://phabricator.kde.org/D29462

To: Zren, #kirigami, mart
Cc: plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, 
ahiemstra, davidedmundson, mart

D29548: Drag and drop assigning windows to activities in the default switcher

[Marco Martin]

mart accepted this revision.
mart added a comment.
This revision is now accepted and ready to land.


  What Nate says, after that go for it

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D29548

To: ivan, #plasma, mart, davidedmundson, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29535: Add a SpinBox (with draggability) to panel thickness adjustment UI

[Marco Martin]

mart added a comment.


  -1 :(

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29535

To: ngraham, abetts, #vdg, #plasma
Cc: mart, ahiemstra, cblack, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D29637: Introduce ImageColors

[Carson Black]

cblack added inline comments.

INLINE COMMENTS

> colorutils.h:182
> +/**
> + * Returns the Cielab "Chroma" of the color which is a slightly better 
> quantificator for how much a color appears 
> https://en.wikipedia.org/wiki/Colorfulness
> + * This is how much a color looks diffenent from a gray of the same

Returns the CIELAB chroma of the given color.
  
  CIELAB chroma may give a better quantification of how vibrant a color is 
compared to HSV saturation.
  
  \sa https://en.wikipedia.org/wiki/Colorfulness
  \sa https://en.wikipedia.org/wiki/CIELAB_color_space

> imagecolors.h:64-70
> + * The source which colors should be analyzed, it can be:
> + * * Any Item: it will be rendered to an image and the static grab 
> + * analyzed. It won't be updated if the item changes, but manual 
> + * calls to update() are needed.
> + * * A QImage (for example coming from a QAbstractItemModel data role)
> + * * A QIcon (for example coming from a QAbstractItemModel data role)
> + * * An icon name: an icon name present in the theme.

The source from which colors should be extracted from.
  `source` can be one of the following:
  * Item
  * QImage
  * QIcon
  * Icon name
  
  Note that an Item's color palette will only be extracted once unless you call 
`update()`, regardless of how the item hanges.

> imagecolors.h:75-82
> + * A list of the color palette extracted from the image.
> + * It uses K-means-clustering tecnique, by averaging groups of
> + * "similar" colors https://en.wikipedia.org/wiki/K-means_clustering
> + * it's a list of maps containing the following keys:
> + * * "color": the color of the cluster
> + * * "ratio": the ratio from 0 to 1 of diffusion of the cluster in the 
> image
> + * * "contrastingColor": another color from the clusters (if possible) 
> that is the nearest to its negative

A list of colors and related information about then.
  
  Each list item has the following properties:
  * `color`: The color of the list item.
  * `ratio`: How dominant the color is in the source image.
  * `contrastingColor`: The color from the source image that's closest to the 
inverse of `color`.
  
  The list is sorted by `ratio`; the first element is the most dominant color 
in the source image and the last element is the least dominant color of the 
image.
  
  \note K-means clustering is used to extract these colors; see 
https://en.wikipedia.org/wiki/K-means_clustering.

> imagecolors.h:86-89
> +/**
> + * If true, it should be considered a "Dark" color palette (this if the 
> dominant color is darker than a 50% gray
> + */
> +Q_PROPERTY(bool isDarkPalette READ isDarkPalette NOTIFY paletteChanged)

I think it would be better to have this return a `ColorUtils::Brightness` enum.

> imagecolors.h:92
> +/**
> + * The average color of the whole image.
> + */

The average color of the source image.

> imagecolors.h:97
> +/**
> + * The dominant color of the image. This is the color of the cluster 
> which covers the bigger area of the image
> + */

The dominant color of the source image.
  
  The dominant color of the image is the color of the largest cluster in the 
image.
  \sa https://en.wikipedia.org/wiki/K-means_clustering

> imagecolors.h:102
> +/**
> + * Suggested "contrasting" color to the dominant one. It's the color in 
> the palette nearest to the negative of the dominant
> + */

The color closest to the inverse of the dominant color found in the source 
image.

> imagecolors.h:107-109
> + * An "accent" color extracted from the image, heuristically found most 
> "vibrant" color from the cluster with highest CIELab Chroma
> + * https://en.wikipedia.org/wiki/Colorfulness#Chroma
> + * which is a better indication than saturation as doesn't decrease with 
> lightness

An accent color extracted from the source image.
  
  The accent color is the color cluster with the highest CIELAB chroma in the 
source image.
  
  \sa https://en.wikipedia.org/wiki/Colorfulness#Chroma

> imagecolors.h:114-116
> + * A color suggested for foreground items over the image
> + * * on dark palettes will be closestToWhite if light enough, or a very 
> light gray otherwise
> + * * on light palettes will be closestToBlack if dark enough or a very 
> dark gray

A color suitable for rendering text and other foreground over the source image.
  
  On dark items, this will be the color closest to white in the image if it's 
light enough, or a bright gray otherwise.
  On light items, this will be the color closest to black in the image if it's 
dark enough, or a dark gray otherwise.

> imagecolors.h:120-124
> +/**
> + * A color suggested for background items over the image, like a dialog 
> frame
> + * * on light palettes will be closestToWhite if light enough, or a very 
> light gray otherwise
> + * * on dark palettes will be closestToBlack if dark enough or a very 
> dark gray
> + 

D29535: Add a SpinBox (with draggability) to panel thickness adjustment UI

[Marco Martin]

mart added a comment.


  Sorry, i don't think it's the right path, for several reasons,
  
  - The spinbox look is different from the desktop one, better or worse is 
secondary, but it's actually a sore point of the plasma theme at the moment :/
  - It exposes very directly pixel numbers in the ui, unless it's a graphic 
application, i hate when it happens in an ui and in our software happens too 
much (and no, no user should ever be asked to know what a pixel is)
  - it keeps the window still when the panel resizes, looking kinda broken, but 
worse, making even possible for the panel to cover up the spinbox itself

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29535

To: ngraham, abetts, #vdg, #plasma
Cc: mart, ahiemstra, cblack, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D29548: Drag and drop assigning windows to activities in the default switcher

[Ivan Čukić]

ivan closed this revision.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29548

To: ivan, #plasma, mart, davidedmundson, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D27353: Allow Activity Switcher to move/add windows to activities by drag and drop from the taskbar

[Ivan Čukić]

ivan requested changes to this revision.
ivan added a comment.
This revision now requires changes to proceed.


  Superseeded by https://phabricator.kde.org/D29548 since there was no activity 
for >2 months.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27353

To: wstephenson, #plasma, ivan, davidedmundson
Cc: anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29651: Notifications applet closes itself after removing all notifications while the system tray popup is pinned open

[Eugene Popov]

epopov created this revision.
epopov added reviewers: ngraham, plasma-devel.
Herald added a project: Plasma.
epopov requested review of this revision.

REVISION SUMMARY
  When I pin a Notifications applet and then remove all notifications, the 
Notifications applet closes itself.
  
  BUG: 421344

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29651

AFFECTED FILES
  applets/notifications/package/contents/ui/main.qml

To: epopov, ngraham, plasma-devel
Cc: Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, 
ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, ahiemstra, mart

D29625: Add global shortcut

[Phabricator]

This revision was automatically updated to reflect the committed changes.
Closed by commit R124:63a6340a711d: Add global shortcut (authored by Guo Yunhe 
).

REPOSITORY
  R124 System Settings

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29625?vs=82532&id=82564

REVISION DETAIL
  https://phabricator.kde.org/D29625

AFFECTED FILES
  app/CMakeLists.txt
  app/systemsettings.desktop

To: guoyunhe, #plasma, mart
Cc: ngraham, davidre, ognarb, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29652: Automatically close Clipboard applet after clearing history or deleting the last element

[Eugene Popov]

epopov created this revision.
epopov added reviewers: ngraham, plasma-devel.
Herald added a project: Plasma.
epopov requested review of this revision.

REVISION SUMMARY
  It would be correct to close the Clipboard applet after clearing the history 
or deleting the last element, since there is nothing more to do with the empty 
Clipboard applet (similar to how the Notification applet closes after the last 
notification is removed).
  
  BUG: 421345

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29652

AFFECTED FILES
  applets/clipboard/contents/ui/clipboard.qml

To: epopov, ngraham, plasma-devel
Cc: Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, 
ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, ahiemstra, mart

D29636: [applets/batterymonitor] Fix tooltip main text not updating while plugged in and charging

[Kai Uwe Broulik]

broulik requested changes to this revision.
broulik added a comment.
This revision now requires changes to proceed.


  The tooltip logic was completely rewritten in 5.19. This patch does not apply 
to 5.18.
  Also, D29633 

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29636

To: ngraham, broulik, #plasma, #localization, yurchor
Cc: yurchor, ltoscano, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29636: [applets/batterymonitor] Fix tooltip main text not updating while plugged in and charging

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> batterymonitor.qml:61
> +} else {
> +return i18n("Battery charging, currently at %1%", 
> pmSource.data.Battery.Percent)
>  }

Also, just because it's plugged in, doesn't mean it's charging.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29636

To: ngraham, broulik, #plasma, #localization, yurchor
Cc: yurchor, ltoscano, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29651: Notifications applet closes itself after removing all notifications while the system tray popup is pinned open

[Kai Uwe Broulik]

broulik accepted this revision.
broulik added a comment.
This revision is now accepted and ready to land.


  Thanks
  please phrase the commit message the opposite way. State what the commit 
changes, not what the state was before the commit.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29651

To: epopov, ngraham, plasma-devel, broulik
Cc: broulik, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29644: Select application after it is added

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> main.qml:97
> +id: transition
> +PropertyAnimation {
> +target: components

Sounds like you want `PropertyAction`

REPOSITORY
  R119 Plasma Desktop

BRANCH
  addIndex (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29644

To: davidre, broulik, ngraham, #plasma, mart
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, 
fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29632: [kcms/keys] Use better collator settings

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> davidre wrote in shortcutsmodel.cpp:89
> We typically don't have apps with numbers but I can also add it?

Ah, this is for apps, but yeah, why not...

REPOSITORY
  R119 Plasma Desktop

BRANCH
  collator (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D29632

To: davidre, ognarb, broulik, #plasma, apol
Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, 
zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

D29288: Improve PageRouter API

[Carson Black]

This revision was automatically updated to reflect the committed changes.
Closed by commit R169:db704d344b2c: Improve PageRouter API (authored by cblack).

REPOSITORY
  R169 Kirigami

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29288?vs=81555&id=82567

REVISION DETAIL
  https://phabricator.kde.org/D29288

AFFECTED FILES
  examples/PageRouterWatchedRoute.qml
  src/pagerouter.cpp
  src/pagerouter.h

To: cblack, #kirigami, mart
Cc: mart, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, 
ahiemstra, davidedmundson

D29616: Fix memory errors caused by using dangling pointers to SensorClients in SensorAgent

[Jiří Paleček]

jpalecek updated this revision to Diff 82568.
jpalecek added a comment.


  Move the client deregistering code from SensorBrowserModel to HostInfo
  
  Also remove the clear function. After investigating further, I've
  found out it suffers from the same problem, that it doesn't deregister
  from the hosts it removes. On top of that, it leaves dangling pointers
  in mSensorInfoMap. It doesn't really clear all the other internal
  structures. And it is never used. So I think it's best to delete it.

REPOSITORY
  R106 KSysguard

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29616?vs=82486&id=82568

REVISION DETAIL
  https://phabricator.kde.org/D29616

AFFECTED FILES
  gui/SensorBrowser.cpp
  gui/SensorBrowser.h
  gui/ksysguard.cpp
  gui/ksysguard.h

To: jpalecek, davidedmundson, ahiemstra
Cc: anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29603: Adds custom spacing between task manager icons

[Nathaniel Graham]

ngraham added a comment.


  In D29603#668422 , @martellini 
wrote:
  
  > > Also, it doesn't make sense IMO to expose separate horizontal and 
vertical controls; there should be just one control visible, and it should 
affect the task manager no matter its orientation.
  >
  > If that's the better way I totally agree, but I don't know recall that 
setting. Is there an open issue that I can look into?
  
  
  This thing: F8310217: this thing 
  
  Right now IMO its function is pretty unclear and useless. Adjusting it 
usually doesn't change anything at all, making it look broken. I think we 
should fix this first.
  
  Conceptually, I think by default, the icon size should smoothly scale with 
the panel thickness. Right now it roughly does this, but with hard sizes rather 
than smoothly scaling the icons, as the macOS Dock does. I think that would be 
a better default.
  
  Then, on top of that, we could additionally have the following settings for 
fine-tuning:
  
  - Spacing (narrow or wide)
  - Maximum icon size (or maybe we don't even need this at all?)
  
  How does that sound?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D29603

To: martellini, #vdg, ngraham
Cc: ngraham, niccolove, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

D29595: Add gocryptfs support

[Nathaniel Graham]

ngraham added reviewers: ivan, Plasma.
ngraham added a comment.


  Very cool!

REPOSITORY
  R845 Plasma Vault

REVISION DETAIL
  https://phabricator.kde.org/D29595

To: mpilia, ivan, #plasma
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart