graesslin added inline comments.

INLINE COMMENTS

> workspaceoptions.cpp:56-58
> +    connect(m_ui->showOsd, &QCheckBox::toggled, this, [this] {
> +        changed();
> +    });

why a lambda? It looks like you just could bind it to a signal

REPOSITORY
  R119 Plasma Desktop

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma:_design, graesslin, #plasma
Cc: andreaska, colomar, graesslin, plasma-devel, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas

Reply via email to