Move KCoreAddons QML bindings to KCoreAddons
Open, Needs TriagePublic

Related Objects

StatusAssignedTask
OpenNone
OpenNone
OpenNone
vkrause created this task.Sep 12 2019, 9:26 AM

As a note, I think we don't want to "move" the wrappers exactly, but instead make use of Q_GADGET on the format and user classes.

dfaure added a subscriber: dfaure.Sep 12 2019, 9:35 AM

Sounds good. I certainly would prefer if KCoreAddons would keep depending only on QtCore.

Yes, whatever glue code we can rid of by adapting the C++ API directly is even better.
Regarding dependencies, I understood that we'd make the QML dependencies optional everywhere, but have the bindings in the modules they provide API for nevertheless (to avoid having to depend on everything due to the binding module pulling in everything).

I certainly would prefer if KCoreAddons would keep depending only on QtCore.

I'm afraid, right now, I would still need some sort of QQmlEngine::registerType to make them usable.
That might change in Qt6, it might be something to push for.

alex added a subscriber: alex.Oct 13 2021, 4:26 AM

@dfaure @davidedmundson What is the state of this?

alex moved this task from Backlog to Waiting on KF6 Branching on the KF6 board.EditedOct 28 2021, 2:35 PM

As confirmed on the ML, having QML stuff as an optional dependency is fine.