gvgeo marked 5 inline comments as done.
gvgeo added a comment.
  After some effort, I reached in these conclusions:
  1.Is not that big of a problem to specify width in a layout. The big problem 
is specifying preferredWidth and height in the same item. Breaks the 
dimensions, this is what got me confused.
  2.Item set as property, does not get managed by layout.
  
  There are still some width set in places. That do not respond to 
preferredWidth, minimumWidth or maximumWidth.

INLINE COMMENTS

> BatteryItem.qml:112
> +                    height: implicitHeight
> +                    width: modelData.label ? detailsLayout.leftColumnWidth : 
> detailsLayout.rightColumnWidth
> +                    wrapMode: Text.NoWrap

This does not react to preferredWidth, minimumWidth or maximumWidth. Is it 
really managed by Layout?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D26806

To: gvgeo, #plasma, #vdg, davidedmundson, manueljlin
Cc: broulik, ngraham, manueljlin, davidedmundson, plasma-devel, Orage, 
LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, 
ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, ahiemstra, mart

Reply via email to