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

Reply via email to