Re: Monday meeting notes for 24/1/2022

2022-01-24 Thread Aleix Pol
Here's the ticket to gather topics for the Plasma sprint:
https://invent.kde.org/plasma/plasma-workspace/-/issues/32

On Mon, Jan 24, 2022 at 5:26 PM Marco Martin  wrote:
>
> Jonathan
> * beta is out, beta testing day is done
> * please fix bugs
> * is there any progress on answering the lts question?
> [discussion recap: lts now means we can't do the "last plasma release"
> for 2 years, which is a quite realistinc timeframe anyways, so
> decision is: let's do an lts-lite that can last less than 2 years]
> [Discussion  will continue on ML]
>
> Arjen
> * so I finally merged a set of patches that have been around for a
> while so we don't set default focus for a number of things if that
> would show a virtual keyboard
> * I also did a few small patches during the bug day last thursday to
> fix some "vhi regression" bugs
> * other than that, there was some discussion on the ml about doing a
> virtual sprint february or march, were there any more concrete plans
> there?
> [Discussion: needs doodle]
> [apol: I guess it could also make sense to discuss which topics we
> need to go through]
> [nico: I'd suggest that we collect topics on the ML in advance]
> [apol: maybe we can have an invent issue]
> [invent issue it is]
> [d_ed: one thing we need to do, potentially at a sprint, is tag things
> as "this is an interesting idea we could do" vs "this has to happen by
> 6.0"]
>
> Bhushan
> * So I've been trying to debug the weird plasma mobile issue where
> plasmashell fails to connect to kactivitymanagerd and then doesn't
> show anything.
> * Some people say it is happening after recent qtwayland update
> * But weirdly enough disabling systemd boot also makes it work
> * I'm continuing to debug it today
> [d_ed will look into it as it seems systemd-related]
>
> Nate
> * So I have been doing a lot of bug triaging for the Plasma 5.24 beta
> and we're in a pretty good shape.
> *We're down to only 6 issues in the "VHI regressions" list:
> https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED_status=CONFIRMED_status=ASSIGNED_status=REOPENED=regression%2C%20_type=allwords_name=VHI-priority%20Plasma%20regressions_id=1967106=VHI=Bluedevil=Breeze=Discover=drkonqi=frameworks-kirigami=frameworks-plasma=frameworks-qqc2-desktop-style=kactivitymanagerd=kde-gtk-config=kdeplasma-addons=khelpcenter=kinfocenter=klipper=kmenuedit=krunner=KScreen=kscreenlocker=ksmserver=ksysguard=KSystemLog=kwin=Plasma%20SDK=Plasma%20Vault=Plasma%20Workspace%20Wallpapers=plasma-integration=plasma-nm=plasma-pa=plasma-simplemenu=plasmashell=policykit-kde-agent-1=Powerdevil=print-manager=printer-applet=pulseaudio-qt=systemsettings=Touchpad-KCM=user-manager=xdg-desktop-portal-kde_based_on=VHI-priority%20Plasma%20regressions_format=advanced
>
> [d_ed will take 448475]
> [marco will take 370676]
>
>
> Aleix:
> * maybe most useful to comment is sddm. I'd encourage anyone who can
> to test the develop branch
> * we'd like to get a release out so we can work on simplifying some
> parts of the codebase and given how important a component it is, it
> would be useful
> * I'm aware there's some important pending PRs, we're trying to go
> through them but it's also a delicate beast in itself
> * testing these MRs on the different use-cases we might have would
> also make it easier for reviewers
> * other than that, I've been working on Discover and video recording
> on wayland, not much to comment on that though
>
>
> Kai Uwe
> Dolphin/KIO:
> Tab bar now accepts drop only ontop of a tab (denied cursor on blank 
> space)
> Fix for janky show/hide animation in KFilePlacesView, please
> review: https://invent.kde.org/frameworks/kio/-/merge_requests/725
> "Details" tab from Baloo-widgets is now only shown when it
> actually has some details (it doesn't fetch them for remote locations)
> Did some investigation as to why KFilePlacesView drop on place is
> broken (cf. last week's notes),
> Qt ignore()s the event and subsequently runs into the fallback
> of calling dropMimeData to add the place to the model
> 
> https://code.qt.io/cgit/qt/qtbase.git/tree/src/widgets/itemviews/qlistview.cpp?h=5.15#n963
> No idea what the right fix would be here. Just not calling
> into the superclass in this case also wreaks all sorts of havoc to the
> view's internal state. And doing a check for that special case in
> KFilePlacesModel doesn't work as dropMimeData doesn't get the QEvent,
> just the QMimeData within. And adding a special mime type is also ugly
> because the QMimeData in the QEvent we get is const … whatever I try
> :-(
> Plasma:
> Fixed regression in Bluetooth applet breaking the big "Enable"
> button when it's off
> Other:
> Noticed a breakage in SDDM in neon, identical to
> https://bugs.kde.org/show_bug.cgi?id=441080 – no idea why and how
>
>
> Nico
> * Fixed task manager identification for some XWayland apps, e.g. Chrome 
> webapps
> https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1396
> * which requires 

Monday meeting notes for 24/1/2022

2022-01-24 Thread Marco Martin
Jonathan
* beta is out, beta testing day is done
* please fix bugs
* is there any progress on answering the lts question?
[discussion recap: lts now means we can't do the "last plasma release"
for 2 years, which is a quite realistinc timeframe anyways, so
decision is: let's do an lts-lite that can last less than 2 years]
[Discussion  will continue on ML]

Arjen
* so I finally merged a set of patches that have been around for a
while so we don't set default focus for a number of things if that
would show a virtual keyboard
* I also did a few small patches during the bug day last thursday to
fix some "vhi regression" bugs
* other than that, there was some discussion on the ml about doing a
virtual sprint february or march, were there any more concrete plans
there?
[Discussion: needs doodle]
[apol: I guess it could also make sense to discuss which topics we
need to go through]
[nico: I'd suggest that we collect topics on the ML in advance]
[apol: maybe we can have an invent issue]
[invent issue it is]
[d_ed: one thing we need to do, potentially at a sprint, is tag things
as "this is an interesting idea we could do" vs "this has to happen by
6.0"]

Bhushan
* So I've been trying to debug the weird plasma mobile issue where
plasmashell fails to connect to kactivitymanagerd and then doesn't
show anything.
* Some people say it is happening after recent qtwayland update
* But weirdly enough disabling systemd boot also makes it work
* I'm continuing to debug it today
[d_ed will look into it as it seems systemd-related]

Nate
* So I have been doing a lot of bug triaging for the Plasma 5.24 beta
and we're in a pretty good shape.
*We're down to only 6 issues in the "VHI regressions" list:
https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED_status=CONFIRMED_status=ASSIGNED_status=REOPENED=regression%2C%20_type=allwords_name=VHI-priority%20Plasma%20regressions_id=1967106=VHI=Bluedevil=Breeze=Discover=drkonqi=frameworks-kirigami=frameworks-plasma=frameworks-qqc2-desktop-style=kactivitymanagerd=kde-gtk-config=kdeplasma-addons=khelpcenter=kinfocenter=klipper=kmenuedit=krunner=KScreen=kscreenlocker=ksmserver=ksysguard=KSystemLog=kwin=Plasma%20SDK=Plasma%20Vault=Plasma%20Workspace%20Wallpapers=plasma-integration=plasma-nm=plasma-pa=plasma-simplemenu=plasmashell=policykit-kde-agent-1=Powerdevil=print-manager=printer-applet=pulseaudio-qt=systemsettings=Touchpad-KCM=user-manager=xdg-desktop-portal-kde_based_on=VHI-priority%20Plasma%20regressions_format=advanced

[d_ed will take 448475]
[marco will take 370676]


Aleix:
* maybe most useful to comment is sddm. I'd encourage anyone who can
to test the develop branch
* we'd like to get a release out so we can work on simplifying some
parts of the codebase and given how important a component it is, it
would be useful
* I'm aware there's some important pending PRs, we're trying to go
through them but it's also a delicate beast in itself
* testing these MRs on the different use-cases we might have would
also make it easier for reviewers
* other than that, I've been working on Discover and video recording
on wayland, not much to comment on that though


Kai Uwe
Dolphin/KIO:
Tab bar now accepts drop only ontop of a tab (denied cursor on blank space)
Fix for janky show/hide animation in KFilePlacesView, please
review: https://invent.kde.org/frameworks/kio/-/merge_requests/725
"Details" tab from Baloo-widgets is now only shown when it
actually has some details (it doesn't fetch them for remote locations)
Did some investigation as to why KFilePlacesView drop on place is
broken (cf. last week's notes),
Qt ignore()s the event and subsequently runs into the fallback
of calling dropMimeData to add the place to the model

https://code.qt.io/cgit/qt/qtbase.git/tree/src/widgets/itemviews/qlistview.cpp?h=5.15#n963
No idea what the right fix would be here. Just not calling
into the superclass in this case also wreaks all sorts of havoc to the
view's internal state. And doing a check for that special case in
KFilePlacesModel doesn't work as dropMimeData doesn't get the QEvent,
just the QMimeData within. And adding a special mime type is also ugly
because the QMimeData in the QEvent we get is const … whatever I try
:-(
Plasma:
Fixed regression in Bluetooth applet breaking the big "Enable"
button when it's off
Other:
Noticed a breakage in SDDM in neon, identical to
https://bugs.kde.org/show_bug.cgi?id=441080 – no idea why and how


Nico
* Fixed task manager identification for some XWayland apps, e.g. Chrome webapps
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1396
* which requires extending the plasmawindowmanagement protocol, so
lots of accompaning MRs
* Besides that I ported some things to the new ecm_add_qml_module
* I made patches to show pairing notifications for bluetooth and KDE
Connect even when in Do not Disturb, because those require user
interaction
* https://invent.kde.org/plasma/bluedevil/-/merge_requests/71
*