KXmlGui
Open, Needs TriagePublic

cullmann moved this task from Needs Input to Backlog on the KF6 board.Nov 24 2019, 9:30 AM
nicolasfella renamed this task from KXmlGui Cleanup to KXmlGui.Nov 25 2019, 9:25 PM
nicolasfella moved this task from Backlog to Metatasks on the KF6 board.

Here's a graph of the current dependencies:

There's quite a bit of optimization potential here. In particular the natural candidates to go after would be KIconThemes and KConfigWidgets.

KConfigWidgets is mostly used for KStandardAction, T12148 discusses moving that to a Tier 1 location.
It is also needed for KLanguageButton, moving that to KWidgetsAddons is discussed in T12197

KIconThemes is used for:
KIconDialog, moving that to Tier 1 is discussed in T14296
KIconTheme to query some sizes: https://invent.kde.org/frameworks/kxmlgui/-/blob/master/src/ktoolbar.cpp#L332
Connecting to iconLoaderSettingsChanged, does that even do something useful? https://invent.kde.org/frameworks/kxmlgui/-/blob/master/src/ktoolbar.cpp#L265
Querying some size in https://invent.kde.org/frameworks/kxmlgui/-/blob/master/src/ktoolbar.cpp#L1058

The Attica dep is discussed in T12198

nicolasfella added a comment.EditedApr 23 2021, 11:58 AM

This is how it would look without KConfigWidgets and KIconThemes: