monday meeting notes for 27/04/2020

[Marco Martin]

Bhushan:
* Gitlab migration coming!
* https://mail.kde.org/pipermail/kde-community/2020q2/006194.html
* and that's it I suppose from my side, not much of direct plasma
stuff this week

Kai Uwe:
plasma-browser-integration:
* Supports Brave browser now
    * Technically supports any Chromium but now identifies as "Brave"
to the OS rather than generic Chrome
* Doesn't offer sending non-htttp(s)/ftp links via KDE Connect anymore
    * Can't send around `blob` URLs :)
Notifications:
* Fixed erroneously grouping notifications with different origins
Other:
* Some more KRun and KToolInvocation porting
* KWindowSystem now prints a meaningful warning when there is no QGuiApplication
* Dropped kded's delayed second phase, loads everything at once now
    * KDED used to load "essential" modules first, then plasma startup
continued, and at the end the rest was loaded
    * Turns out this doesn't really gain us much but added complexity
makes it hard to replicate in sytemd unit dependency tree
* Wrote an ECM module for generating a dbus service definition, please
review: D29051
    Diff 29051 "Add ecm_generate_dbus_service_file" [Needs Review]
https://phabricator.kde.org/D29051
Personal:
* I'll be leaving Blue Systems at the end of next month, so please
expect my responsiveness in reviews and bug reports to reduce in the
months thereafter

David E:
* I need some opinions on my systemd startup patch - it's not ready to
be default, but I'm a huge believer in slow roll outs so I'd like to
merge it even though it's off and build up that wider testing
(Note: Opinions say yes)
* it still installs all the .service files
* but no-one ever invokes them
* in other (somewhat worrying news) I have a crash on latest Qt5.15
randomly in QString
* https://bugs.kde.org/show_bug.cgi?id=420452
    KDE bug 420452 in plasmashell (Task Manager) "Crash on load in
taskmanager" [major,] https://bugs.kde.org/show_bug.cgi?id=420452
* I can easily work round this porting to a loop, but Aleix is hitting
the same crash somewhere else
* will try to make a minimal test case
* In other news, I've been doing lots of wayland stuff - make sure
everyone sees the result of the ML thread "Problems in KWayland causes
by API and ABI compatibility promises" on frameworks devel

Nico:
* Started list of things that need fixing in KDE Connect Flatpak
https://phabricator.kde.org/T13047
* Task 13047 "Improve Flatpak version" [Open,Needs Triage] {KDE
Connect} https://phabricator.kde.org/T13047
* Added device type detection in kdeconnect-kde, PlaMo now shows up as
phone, BigScreen as TV (PLASMA_PLATFORM env var not ideal, but better
than nothing)
* $dayjob project but still somewhat KDE: clazy now can check for
invalid JNI signatures when doing Android things
* Ported KTrip to date/time controls from kirigami-addons
* Fixed sizing issues in said controls
* Started upstreaming my date/time picker dialogs with Android support
into kirigami-addons, but got stuck on first weird Android issues and
now weird QML issues


David R:
* SO fixed a bug in breeze where icons on comboboxes where colored
wrong when hovered and focused
* https://phabricator.kde.org/R31:5a62232415bc3776989474faa6f4efb2e0c05d1a
* Set a componentDisplaynames on the action colelctions of some kcms
so that the registered shortcuts don't show up as "KDE Daemon" because
the code is executed by the kded process
* https://phabricator.kde.org/R120:9056449edc892e88788f189c0306c821c4b87e4d
* https://phabricator.kde.org/R119:e64a3cb2af230cc3d528a45c5e7f80db74b09540
* https://phabricator.kde.org/D29154
* Diff 29154 "Use QWindow overload of QIcon::pixmap" [Needs Revision]
https://phabricator.kde.org/D29154
* Previously only the comboboxlabel used this overload. Seems also
that we don't get a repaint when the dpi changes
* And in the screenshots apol posted somehow the sidebar of dolphin
seems blurry with master and this patch. So if you have highdpi or
mixed dpi set ups please test :) (this is probably phab doing things
to the image)
* anyways style not reacting correctly to dpi changes, needs more digging
* Unrelated but just remembered:
* Did you see the question about kiconloader signals and kcm on ml?
https://mail.kde.org/pipermail/kde-frameworks-devel/2020-April/109732.html
* I think kbroulik is responisble :)
* It seems the kcm fails to emit a signal when some settings are changed

Marco:
# Plasma
* commandline parameter to load a single module with Systemsettings to
eventually replace kcmshell D29064
* services runner loads kcms with systemsettings D29157
* Needs discussion: can kcmshell5 really be killed? have a symlink
kcmshell5->systemsettings5 or change the Exec= line of *every* kcm?
* Needs discussion: T10201 i think it needs a KColorScheme expansion
to work "properly": opinions?
* continuing work on the systemmonitor applet refactor, now the faces
library is a separate lib and import in the libksysguard repo, all
faces have been moved there and ported.
# Kirigami
* some fixes on overlaysheet