Minutes meeting week 24

Hello people, Sorry for delay in sending this, attached are notes and logs romangg: * pushed D13191 (not in 5.47) * more work on pointer constraints: * added a test: D13439 * bug fix: D13466 * need review: * D13257 * also needs review: D13084 * My plan for pointer constraints is to

D13255: Make dependency on KF5NetworkManagerQt optional

ivan accepted this revision. ivan added a comment. This revision is now accepted and ready to land. Thanks! REPOSITORY R845 Plasma Vault BRANCH arcpatch-D13255 REVISION DETAIL https://phabricator.kde.org/D13255 To: asturmlechner, #plasma, ivan Cc: plasma-devel, ragreen, Pitel, ZrenBot

D13178: Compress calls to configuration slots upon new connections

jacopods updated this revision to Diff 36031. jacopods added a comment. I used the direct connection as suggested. Today I learned about QOverload... CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D13178?vs=35045&id=36031 REVISION DETAIL https://phabricator.kde.org/D13178 AFF

D13487: Fix build against static Qt without QLibrary

dvratil created this revision. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. dvratil requested review of this revision. REVISION SUMMARY Sometimes when Qt is built statically it is also built without support for QLibrary and friends,

D13481: Recommend window border size "None"

romangg created this revision. romangg added reviewers: Plasma, VDG. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. romangg requested review of this revision. REVISION SUMMARY Adds a recommendation of border size "None" for the new KWin m

D13372: Mouse KCM Redesign Using Kirigami

furkantokac added a comment. In D13372#277078 , @mart wrote: > here a patch against this which ports it to qqc2, using the completely different radiobutton mechanics qqc2 uses, fixing separators, removing plasmacore import > F5907175: qqc2.di

D13372: Mouse KCM Redesign Using Kirigami

mart added inline comments. INLINE COMMENTS > main.qml:105 > > -Layouts.ColumnLayout { > -id: maincol > -enabled: deviceCount > -spacing: units.largeSpacing > +Kirigami.Separator { > +} this is broken, it needs Kirigami.FormData.isSec

D13372: Mouse KCM Redesign Using Kirigami

mart added a comment. here a patch against this which ports it to qqc2, using the completely different radiobutton mechanics qqc2 uses, fixing separators, removing plasmacore import F5907175: qqc2.diff REPOSITORY R119 Plasma Desktop REVISION DETAIL

D13372: Mouse KCM Redesign Using Kirigami

mart added inline comments. INLINE COMMENTS > main.qml:27 > +import org.kde.kirigami 2.4 as Kirigami > import org.kde.plasma.core 2.0 as PlasmaCore > never import anything from plasma in a kcm (an exception which we have to solve is SortFilterProxyModel) Kirigami has its own units > main.qm

D13372: Mouse KCM Redesign Using Kirigami

mart added a comment. In D13372#277047 , @furkantokac wrote: > In D13372#277040 , @mart wrote: > > > for me, using qqc1 in a new kcm, is an absolute blocker, no matter what the problem is with qqc2,

D13372: Mouse KCM Redesign Using Kirigami

furkantokac added a comment. In D13372#277040 , @mart wrote: > for me, using qqc1 in a new kcm, is an absolute blocker, no matter what the problem is with qqc2, problems must be fixed, not worked around. ever. So please keep following the

D13372: Mouse KCM Redesign Using Kirigami

mart requested changes to this revision. mart added a comment. This revision now requires changes to proceed. for me, using qqc1 in a new kcm, is an absolute blocker, no matter what the problem is with qqc2, problems must be fixed, not worked around. ever. REPOSITORY R119 Plasma Desktop RE

D13476: Remove doxygen docs of internal class

davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. davidedmundson requested review of this revision. REVISION SUMMARY And copy the relevant snippet to the releva