[Differential] [Commented On] D1771: [Workspace Options] Add option to disable OSD

[Martin Gräßlin]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170109/98dd640a/attachment.html>