monday meeting notes for 16/11/2020

[Marco Martin]

Carl
* I made a bunch of MR in discover, most are approved and also two MR
in kirigami
* https://invent.kde.org/frameworks/kirigami/-/merge_requests/153
* https://invent.kde.org/frameworks/kirigami/-/merge_requests/165
* other than that, I just continued working on Neochat (a kirigami
matrix client)

DavidR
- Fix ctrl+q only working on second press in plasma-systemmonitor
https://invent.kde.org/frameworks/kirigami/-/merge_requests/149
- Search runtime dependencies of plasma-systemmonitor in cmake
https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/70
- Fix plasma-systemmonitor bugs that were reported via bugzilla
https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/67
- Do not add unnecessary seperators in drop menu
https://invent.kde.org/frameworks/kio/-/merge_requests/209
- Lock the screen when connecting to logind (for example when kwin is
restarted while the session is locked)
https://invent.kde.org/plasma/kscreenlocker/-/merge_requests/14
- Align fake titlebar in colors kcm with the real code
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/447
- Fix size of cursor sizes previews
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/449
- Merge keysequence patch series \o/
Currently working on documentation for libksysguard, sensors and faces libraries

Kai Uwe
Notifications:
* Looked into using CGroups for identifying sending application
    * when run from Terminal it will inherit Konsole..
    * Preparatory patch, please review
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/406
* Fixed pause button not indicating job paused state, was PC3 porting regression
* Worked on not offering "Open" when moving to trash:/ but fighting
with some odd QQC Layout issues
KRunner:
* Location runer: look explicitly for URL scheme handlers, please
review https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/455
    * Currently only does `KProtocolInfo` checks which only knows KIO protocols
    * With this patch you can also type URLs, like rtmp:// to open in
video player or spotify:/

Fabian
* My MR for KConfig::Notify for kdeglobals in breeze got accepted and
openQA is happy with that
* Dolphin's URL bar moved, which is not only much less usable, but
also broken (not Plasma, but maybe someone has input)
* Breeze renders a gap inside frames now,
https://invent.kde.org/plasma/breeze/-/merge_requests/52
* Systemsettings shows the appearance sub-category in a broken way: -
systemsettings:
https://invent.kde.org/plasma/systemsettings/-/merge_requests/32#note_136280
[there was a discussion about the systemsettings change, needs further
discussion with VDG]


Nico
* There was a Qt change that caused some build fallout where raw chars
were used. Fixed by using QLtin1Char
* Add frame around PC3 scrollview background to be consistent with
qqc2-desktop-style
https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/144
* Add frame around PC3 scrollview background to be consistent with
qqc2-desktop-style
https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/144
* Make nextcould window small enough for mobile
https://invent.kde.org/network/kaccounts-providers/-/merge_requests/20
* Remove the useless SMB KCM
https://invent.kde.org/frameworks/kio/-/merge_requests/212
* Use Kirigami.PasswordField
https://invent.kde.org/network/kaccounts-providers/-/merge_requests/22
* clang-format kaccounts-providers and kaccounts-integration
https://invent.kde.org/network/kaccounts-integration/-/merge_requests/18
https://invent.kde.org/network/kaccounts-providers/-/merge_requests/23
* Fix accounts KCM blocking quitting system settings when config was cancelled
* Some cleanup in KDE Connect
* Removed useless autostart path from paths KCM
https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/221
* Fix Scrolling with PC3 scrollview
https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/137


Arjen
* I've mostly been busy with some smaller bugfixing/feature work for
system monitor
* I started work on changing the ksystemstats update rate to 500ms
instead of 2000
* but then realised that the line chart needs a different setting for
its X axis, so I started working on changing it to "amount of history
to keep in seconds"
* and then I got distracted by a bug report about adding a context
menu to the global drawer items... which turns out to be a lot more
involved than intended
* so I have a bunch of stuff in progress but nothing complete atm
* the line chart work is partially in preparation for having different
update rates for different sensors


David E
* In terms of things to discuss, I found out why krunner looks so
jumpy, it's a bug in dialog having a queued event when item sizes
changes. So I finally started a "Dialog2"
* Intention to have a very simple class that draws a frameSVG and
positions a main item, then do everything else (like popup
positioning) as subclasses or decorators
* in the new one implicit size propagates up to window, actual size
propagates down to client
*  Other thing I wanted to discuss, is the list of "V-high plasma
bugs" is growing, so I want to run through some of them
  KDE bug 428613 in kscreenlocker (general) "Doesn't communicate to
the user what's going on when the session has been locked due to
excessive password attempt when pam_deny is in use" [normal,]
https://bugs.kde.org/show_bug.cgi?id=428613
** I started a draft patch to kscreenlocker to have an in-process PAM
object in a thread, so we have more direct access to the full
conversation https://invent.kde.org/plasma/kscreenlocker/-/merge_requests/15
** so we can introduce it to do finger print unlocking which I've also
seen some work towards for 5.21
* BUG 424408
  KDE bug 424408 in plasmashell (general) "Multiple coredumps with
every login/logout" [crash,]
https://bugs.kde.org/show_bug.cgi?id=424408
** it seems we relied on ksmserver to close down a lot of
applications, so my patchset to disable session management was a
really stupid thing
** it seems we relied on ksmserver to close down a lot of
applications, so my patchset to disable session management was a
really stupid thing
** which used to gracefully close down the other half of stuff
** systemd session unaffected, only the legacy path
** there is a hack for the dbus daemon issue, which is copied from gnome
** we can explicitly kill dbus-daemon on logout
** which sounds horrific...but gnome do it, so how bad can it be
** revert my disable session management patches (done most, but some
others remain)
** explicitly quit kwin (patches on review)
[fvogt:  FTR, I filed https://bugreports.qt.io/browse/QTBUG-88491 to
avoid the coredumps on logout from the other side]
*  BUG 408116  I fixed, was just a minor code bug. See link in report
   KDE bug 408116 in systemsettings (kcm_touchpad) "[libinput] Options
to set right-click are greyed out and middle-click options when using
Icons view in System Settings, or in the KCMShell version, or when on
Wayland after opening and closing the KCM in SystemSettings a few
times" [major,] https://bugs.kde.org/show_bug.cgi?id=408116
* the rest of the v-high bugs need some attention, so please do take any

Marco
# Plasma
* exposing the containment type in appletinterface broke folderview,
so now there is a new trickle down property: containmentDisplayHints,
which is a flag, so is expandible with more combination of values in
the future
* systemtray uses containmentDisplayHints
* ported weather applet to containmentDisplayHints
* fixed last issue in the new systemsettings hyerarchy stuff, merged \o/
# Firewall KCM / Kirigami
* title for the add rule sheet now in the header
* Fix a bug in kirigami that prevented the "add new rule" sheet to
display at the corect position
# RTL fixes
* Fixed Kirigami drawer handles in RTL mode
* fixed qqc2-desktop-style RangeSlider in RTL mode
* fixed plasma qqc2 style sliders in RTL mode
* fixed plasma qqc2 style mobile scrollbars in RTL mode
# Plasma Mobile
* fixed an issue on top panel opening the first time
# System monitor
* doing a new face for the store