D24519: Delegate KCM cursor theme config management to KConfigXT

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> main.qml:99
>                      textRole: "display"
> +                    currentIndex: kcm.cursorSizeIndex(kcm.cursorThemeSettings.cursorSize);
>                      onActivated: {

I think this will be broken if you change this combo box, then change theme.

When a user explicitly changes this combo box this binding gets broken with currentIndex now reflecting an explicit number.

On theme change cursorThemeSettings.cursorSize changes correctly but our binding is gone.

This might have to be a connection again

REPOSITORY
  R119 Plasma Desktop

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

To: bport, #plasma, mart, ervin
Cc: davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191009/a297e9fd/attachment.html>