D12946: Single/Double Click Option at Workspace KCM

[Roman Gilg]

romangg added inline comments.

INLINE COMMENTS

> main.qml:82
> +
> +                Controls.RadioButton {
> +                    id: doubleClick

Add: `checked: !kcm.singleClick`

> furkantokac wrote in main.qml:90
> This is not okay. If singleclick is not set initially, doubleclick radio button should be enabled. This code doesn't to that.

singleClick will be set initially through https://phabricator.kde.org/D12946#inline-66775

But you are right, it also needs `checked: !kcm.singleClick` for doubleClick to work.

REPOSITORY
  R119 Plasma Desktop

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

To: furkantokac, romangg, ngraham, #plasma
Cc: davidedmundson, plasma-devel, 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/20180517/12cfd179/attachment.html>