D25000: Do not assign combobox currentIndex as it breaks binding.

Cyril Rossi noreply at phabricator.kde.org
Wed Oct 30 13:16:11 GMT 2019


crossi updated this revision to Diff 69053.
crossi added a comment.


  Remove assignement to currentIndex.
  Add a C++ bypass to setCurrentIndex without breaking the binding.

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D25000?vs=68874&id=69053

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

AFFECTED FILES
  org.kde.desktop/ComboBox.qml
  plugin/CMakeLists.txt
  plugin/kpropertywriter.cpp
  plugin/kpropertywriter.h
  plugin/qqc2desktopstyleplugin.cpp

To: crossi, #plasma, ervin, mart, bport, broulik
Cc: 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/20191030/6f3a4191/attachment.html>


More information about the Plasma-devel mailing list