Skip to content

[ExpandableListItem] Load expanded view on demand

Kai Uwe Broulik requested to merge work/expandable-load-on-demand into master

I noticed that plasma-nm would create all connection details immediately and then I found ExpandableListItem would always load the full representation which is a huge waste. Also, binding to visible can have unintended side-effects when nested (it is inherited from its parent).

Instead, bind everything to active to load and unload the expanded view on demand.

  • Plasma-nm does not load connection details for all views immediately anymore
  • Device notifier expanding of devices still works

Note that this will cause customExpandedViewContentItem to be null when the item isn't expanded but the documentation already says "The actual instance of the custom view content, if loaded" so we merely make this work like this now :)

Question: The list item does not check for whether there are actions or a custom component, right? So the expand arrow will always show, right?

Merge request reports