----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/#review75012 -----------------------------------------------------------
src/kded/kdedactions.cpp <https://git.reviewboard.kde.org/r/122319/#comment51940> Use qWarning rather than qDebug, I'd say it's a warning after all... src/kded/kdedactions.cpp <https://git.reviewboard.kde.org/r/122319/#comment51941> If this is not needed anymore, just remove it altogether - Martin Klapetek On Jan. 29, 2015, 11:56 p.m., Rajeesh K Nambiar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122319/ > ----------------------------------------------------------- > > (Updated Jan. 29, 2015, 11:56 p.m.) > > > Review request for Plasma, Alexander Mezin and Jonathan Riddell. > > > Bugs: 342629 > http://bugs.kde.org/show_bug.cgi?id=342629 > > > Repository: kcm-touchpad > > > Description > ------- > > Set global shortcuts for touchpad enable/disable/toggle. > Review and comments welcome. > Bug 342629 > > > Diffs > ----- > > src/kded/kdedactions.cpp ca32834 > > Diff: https://git.reviewboard.kde.org/r/122319/diff/ > > > Testing > ------- > > Build Fedora RPMS against Plasma 5.2, default global shortcuts for touchpad > are set. > > > Thanks, > > Rajeesh K Nambiar > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel