Monday meeting notes for 15/2/2021

[Marco Martin]

Kai Uwe
Notifications:
* Removed `QImageItem` for notification icon, Plasma `IconItem` seems
to work well for `QImage`, tooltips
* Worked on calculating transfer speed in Notifications
    * KIO hardly ever reports a speed unless copying large files
because it does it on a per-file basis
    * so I thought I can do it on notification applet side where I
have the overall processed bytes
* Will change the new `KUiServerV2JobTracker` to to the delay in the
job tracker side
    * Since it's already async and remembers job state until it is
registered should be easy to do the delay there, too, without missing
any update
    * Avoids requesting a job view on DBus for short-lived states that
we wouldn't display anyway
Other:
* Was investigating blocking DBus calls in plasma-nm, turns out Qt
DBus property handling is even worse than I thought
    * a generated setFoo() method for a property will do
setProperty("Foo") which will then block to return true/false on the
setProperty call, even though setFoo doesn't care
        * This way it blocks 250ms here when turning off wifi, and 4ms
every time I open the applet when it changes the update interval...
* Ported Styles KCM away from `QDBusInterface`
* Port BluezQt away from `QDBusInterface`, please review:
https://invent.kde.org/frameworks/bluez-qt/-/merge_requests/6
* Bluez blocks quite some time in rfkill handling when turning on/off Bluetooth
    * it opens the `/sys/class/rfkill/...` with `O_NOBLOCK` but I
guess that still blocks on a `write` to a "normal" file?
* `KPropertiesDialog` now shows MIME type name as tooltip for file type


Fabian
* The ksmserver logout issue turned out not to be ksmserver at all, as
it didn't even reach that
* Apparently ksmserver-logout-greeter exits before dbus-activation
finishes, so plasma-shutdown doesn't get the method call to logout at
all (https://bugs.kde.org/show_bug.cgi?id=432460)
    KDE bug 432460 in ksmserver (general) "Logout sometimes doesn't do
anything" [normal,] https://bugs.kde.org/show_bug.cgi?id=432460
    [d__ed currently following up on this]
*  I guess I can also mention my kwin Xwayland MR,
https://invent.kde.org/plasma/kwin/-/merge_requests/681 + some
discussion on related MRs
* openQA for 5.21.0 is at this point almost green, but there are two
remaining issues:
    1. In the wayland session, sometimes pressing "enter" in dolphin
opens the "new folder" dialog instead of entering a folder I have
absolutely no idea why, but that also started happening with 5.20.90
first
     I completely fail to reproduce that locally
     https://openqa.opensuse.org/tests/1629680#step/dolphin/9 presses
enter, then https://openqa.opensuse.org/tests/1629680#step/dolphin/10
happens
    2. On upgrades, the color scheme gets changed to breeze light
     I haven't debugged in detail why the LNF default isn't taken on
upgrades, but I suppose it's the order of kconf_upgrade scripts
     My theory is that on fresh installs, the breeze light migration
runs first, but then gets overwritten by the LNF import (by some other
kconf upd I forgot the name of)


DavidR
- An improvement for settings desktop names on session restore came to
my mind https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/638
- Fixed another crash in breeze
    https://invent.kde.org/plasma/breeze/-/merge_requests/
- Fixed Animations being disabled on start in gtk applications
    https://invent.kde.org/plasma/kde-gtk-config/-/merge_requests/24
- Fixed disk speeds in plasma-systemmonitor
    https://invent.kde.org/plasma/ksysguard/-/merge_requests/66
- Fixed kscreen osd
    https://invent.kde.org/plasma/kscreen/-/merge_requests/13
- Did some work so we pars /proc files only one time in one process in
libksysguard
    https://invent.kde.org/plasma/libksysguard/-/merge_requests/119
- Did some work so we pars /proc files only one time in one process
in libksysguard
    https://invent.kde.org/plasma/libksysguard/-/merge_requests/11
- Tried to see why sometimes the logout greeter apepars in the
taskmanager, no results yet
- Started looking into supporting DbusActivation it in ApplicationLauncherJob

Carl
* I started working on calendar components for Plasma and Plasma
Mobile. For the moment I only have a monthview with events fetched
from a KCalendarCore but I'm working on a weekview with a timeline of
events. There is a performance regression In Kirigiami and NeoChat is
particularly affected :( And also GSoC is coming, does Plasma want to
participate? https://invent.kde.org/carlschwan/kalendar
https://matrix.org/_matrix/media/r0/download/kde.org/f951745c4b1118933cff20a56cce0d0eeadf2de4/image.png
[ discussion on moving it into kirigami-addons]

DavidE
* I've been doing some Qt6 wayland work
* fixing a lot of drag and drop issues, and some other misc things
* all exciting stuff .. for use in 2 years


Bhushan
* So I had been fixing lot of kdereview comments in plamo related
repos earlier today
* I will continue more of that today, and once that is done will move
them to extragear

Jonathan
* tomorrow it's 5.21 day!
* I have to respin  kwin, ksysguard and plasma-desktop
* announce is being led by Carl Schwan
* everything is awesome [Carl: announcement video isn't finished yet,
in progress]
* shall I organise a 5.22 kickoff meeting once it's out?
[there is consensus]

Arjen
* the biggest thing is that I landed the PlatformTheme rework
* apparently with a bad performance regression
** for the platformtheme thing I am changing things to use custom
QEvents, which should make it possible for subclasses to distinguish
what exactly changed and thus greatly reduce the number of times
colors are being reapplied
* other than that I did some plasma-systemmonitor work and made some
fixes to the pie chart when used in a panel

Marco
# Plamo
* More work on the Launcher refactor to make it more behaviorally
similar to Android. ended up quite monumantal, taken ~99% of the week,
but is done.
# Plasma
* plamo homescreen fixes uncovered an upstream bug in the desktop
plasmoid layouting code, fixed and backported
# Sytem Settings
* First round of reviews for the landing page