monday meeting notes for 2/11/2020

[Marco Martin]

Nico
* Most of my work was on plasma-pa:
* The KCM got a look with multiple pages instead of tabs and more
standard list headers.
* https://invent.kde.org/plasma/plasma-pa/-/merge_requests/10
* https://invent.kde.org/plasma/plasma-pa/-/merge_requests/24
* https://invent.kde.org/plasma/plasma-pa/-/merge_requests/25
* https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/208
* There's still the issue that multipage KCM don't work in the applet
config, but after this cleanup it should be a lot easier to implement
* Show a message with the result URL when sharing something via imgur
in Gwenview: https://invent.kde.org/graphics/gwenview/-/merge_requests/21
* Remove some dead code in Kirigami:
https://invent.kde.org/frameworks/kirigami/-/merge_requests/136
https://invent.kde.org/frameworks/kirigami/-/merge_requests/137
* Rework the KCM paddings to work better on Mobile:
https://invent.kde.org/frameworks/kdeclarative/-/merge_requests/30
https://invent.kde.org/frameworks/kdeclarative/-/merge_requests/31
* kaccounts: Display account creation errors in the UI
https://invent.kde.org/network/kaccounts-integration/-/merge_requests/16
* Add windowId to kwin effect API:
https://invent.kde.org/plasma/kwin/-/merge_requests/410

David E
* I've been trying to tackle the "VHI plasma bugs" each weekend, fixed:
 - a crash in kirigami when accessibility is enabled (on review)
 - hopefully all the shutdown/switch user options being missing issues
* This also shows no devs run with accessibility enabled, which is
quite telling in a bad way
* The crash was hit every time after I set it up again
* In terms of other stuff: I fixed an obscure kwin freeze, a dnd
crash, hopefully closed the ksysguard memory leak, and fixed the
stupid startup notification sound

Kai Uwe
plasma-browser-integration:
* Worked on support for Microsoft Edge
    * Technically "just works" but otherwise thinks it's just a generic Chromium
* Made a PoC for using CGroups to identify the browser instead of a
heuristic based on user agent
    * https://invent.kde.org/plasma/plasma-browser-integration/-/merge_requests/17
    * Works in principle (safe for the ugly RegExp stuff, which I'd
like in a reusable lib somewhere)
    * but has the quirk that when running it from terminal it will
inherit Konsole...
* Improved tabs runner favicon handling, please review
https://invent.kde.org/plasma/plasma-browser-integration/-/merge_requests/16
Notifications:
* Also tried using CGroups for identifying apps that don't properly
identify to us
    * Works well for broken notifications like Spotify or snaps but
also obv has the same run from terminal issue
* Added "Move to Trash" / "Delete" entry to File menu so you can
quickly get rid of an unwanted screenshot again
* Noticed that Breeze menus suffer from missing transparency
(WA_TranslucentBackground) a lot more lately
    * Made a workaround in Notifications but I think there's a
regression somewhere:
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/404
    * It never fully worked but it definitely wasn't broken to the
extend it is now
PowerDevil:
* Got rid of xcb dpms timeout handling, it's now done explicitly like
on Wayland, makes the codebase more predictable
Plasma:
* Added setting for first day of week to digital clock calendar
    * Similar to how we have an override for 12/24 hour
* System Tray now falls back to tooltip title if SNI has no title set
* Noticed the EnergyInfo KCM is broken due to a regression in Kirigami
FormLayout, please fix https://bugs.kde.org/show_bug.cgi?id=428533
KDE bug 428533 in frameworks-kirigami (general) "FormLayout
twinLayouts causes loop in energy info KCm" [crash,]
https://bugs.kde.org/show_bug.cgi?id=428533
Other:
* Had ksgrd_network helper go berserk (2 processes both taking 100% of
a core) yesterday while downloading Steam game updates but didn't
investigate/reproduce


Bhushan
- I discussed some details about grab_keyboard with the d_ed earlier this friday
- And I did bunch of fixes in PlaMo sms app


David R
- Only look for libnl
https://invent.kde.org/plasma/ksysguard/-/merge_requests/48
- Do not load multiple versions of the same plugin
https://invent.kde.org/plasma/ksysguard/-/merge_requests/50
- Only allow multi key shortcuts for standard shortcuts
https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/207
- Fix Khotkeys not importing files anymore
https://invent.kde.org/plasma/khotkeys/-/merge_requests/6 ,  Needs review
 Which is a porting bug also in 5.20
- Spend a bunch of time investigating the Qmenu issue (see above discussion)
-yeah I had to do the same when porting knotification notifyrc finding
to a manual loop
- KeySequenceRecorder patch series should be ready and is targeteted
for after next frameworks tagging
- d_ed reviewed the kguiaddons, and kxmlgui mrs, would need a look at
the kdeclarative one too
- https://invent.kde.org/frameworks/kdeclarative/-/merge_requests/27
- I wanted to mention that we have
https://bugs.kde.org/show_bug.cgi?id=427311 caused by the recent
breeze patches

Marco
# Plasma
* fixed smoketest autotest for kdescendantproxymodel
* Systemtray can have 2 context dependent buttons in the title:
configure "current" plasmoid and an hamburger menu of extra contextual
actions
* more work on systemsettings category-owners kcms
* open mrs for those 2 things
https://invent.kde.org/plasma/systemsettings/-/merge_requests/32
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/362
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/374
* experimenting on a "config dialog" qml component (inspired but not
too much from plasmoids config dialogs), for now in kirigami-addons
branch, will probably need discussion
# Plasma Mobile
* fixed thumbnails in task switcher, they now work on pinephone