gvgeo added inline comments.

INLINE COMMENTS

> Advanced.qml:33
> +    // TODO Fix contentview width, and Remove policy.
> +    ScrollBar.horizontal.policy: ScrollBar.AlwaysOff
>  

> What if there's too many devices?

No matter how many devices there are, column is always a column.

> It doesn't sound like a good solution to hide the ScrollBar forever.

That's why a TODO comment exist.

> Also what does contentview refer to?

The width of the content. Need to remove "view".

ContentWidth calculates most of the text with contentWidth instead of 
paintedWidth.
Have not found the loop.

REPOSITORY
  R115 Plasma Audio Volume Applet

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

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

Reply via email to