D27577: [KCM]Fix content below scrollbars

[George Vogiatzis]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200223/e644e349/attachment.html>