D18299: ComboBox: fix default delegate

Kai Uwe Broulik noreply at phabricator.kde.org
Wed Jan 16 15:30:47 GMT 2019


broulik added inline comments.

INLINE COMMENTS

> ComboBox.qml:58
> +            controlRoot.currentIndex = index;
> +            controlRoot.popup.visible = false;
> +        }

Are you sure this is needed? Qt docs say for that using `ItemDelegate` for a `ComboBox` is recommended as:
"This ensures that the interaction works as expected, and the popup will automatically close when appropriate. "
The `currentIndex` change makes sense, though

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

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

To: apol, #frameworks, davidedmundson
Cc: broulik, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190116/d2f8c8f6/attachment.html>


More information about the Plasma-devel mailing list