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
