Code review comment for lp:~unity-team/unity8/unity8.previews_textSummary

Still needs reply:

>> =====
>>
>> 321 + implicitHeight: titleLabel.visible ? titleLabel.height +
>> textLabel.height : textLabel.height
>>
>> childrenRect.height no good?
>>
>
> Visibility doesn't affect childrenRect

Visibility doesn't, but you're changing anchors based on visibility, and
that does affect childrenRect.

>> =====
>>
>> 514 + var mappedTextLabel = root.mapFromItem(textLabel, 0, 0)
>> 515 + compare(mappedTextLabel.y, titleLabel.height)
>>
>> They're in the same geometry, do we really need the mapFromItem?
>>
>
> We're testing the placement now, and then when it disappears and moves on top. Comment added

> 515 + // verify titleLabel is visible and textLabel is anchored below it

Why isn't that simply textLabel.y == titleLabel.height?

http://paste.ubuntu.com/6874503/