Unrelated

Remove

Why this change?

FTR: Qt 5.9 will add a "positioningComplete" signal :)

Can you perhaps have the task do that automatically since fwict you always do forceActiveFocus and then ensureItemVisible:

onActiveFocusChanged: {
    if (activeFocus) {
        ensureItemVisible(this);
    }
}

?

Why not assign it directly above?

property TextMetrics textMetrics: TextMetrics {}

Will do.

Will do.

I was doing heavy refactoring and changing the nesting and trying different things a few times, happened along the way. I'll clean it out of the diff.

Nifty, about time. That also means we can remove some fugly code in main.qml for the layouting and delegate geo publishs.

I see the appeal, but I'd rather not do it implicitly and calling GroupDialog API ... I'm rather unhappy with how much coupling there is between Task and GroupDialog already (writing a delegate that relies on API outside of it is always pretty ugly). I'll think about whether I can find some other nice solution.

Doing it declaratively creates a binding loop in this case.

I think this wasn't possible in older Qt Quicks? Nice that it's possible now, will do.

I don't get how assigning a number as a child works. This line makes no sense to me.

I think this has always been possible :) definitely for years.

What doesn't (until Qt 5.8 which fixes this) is having name spaced declarations, ie. property PlasmaCore.Foo bar will choke on the dot.

Actually, to me it doesn't, either ;). I was asleep at the wheel/rushed earlier, will remove. Thanks for being persistent.

is this timer working around some bug? should have at least some comments explaining why this needs to be delayed