D7733: [Folder View Icons Config] Fix initial index of ComboBox

[Kai Uwe Broulik]

broulik created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  A while ago I optimized Plasma config dialogs to create the component with the cfg_ properties already set. This code relied on the fact that `mode` changes initially. Use `Component.onCompleted` now that we can and also use `onActivated` which only fires if the index is explicitly changed by the user, not on imperative assignment.

TEST PLAN
  Changed sorting to "Name" or "Size", closed dialog, opened it, was still selected. Previously it would always revert to "Unsorted"

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  containments/desktop/package/contents/ui/ConfigIcons.qml

To: broulik, #plasma, hein
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170908/4bc07d50/attachment.html>