> On July 11, 2011, 8:36 p.m., David Edmundson wrote: > > main-widget.h, line 54 > > <http://git.reviewboard.kde.org/r/101918/diff/1/?file=26729#file26729line54> > > > > really try and make an effort to think if something should be public or > > not. > > > > Changing behaviour from the parent (which I assume is predicted) seems > > wrong unless you have a reason.
*protected - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101918/#review4612 ----------------------------------------------------------- On July 11, 2011, 7:44 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101918/ > ----------------------------------------------------------- > > (Updated July 11, 2011, 7:44 p.m.) > > > Review request for Telepathy. > > > Summary > ------- > > This is the second part of the DBus-plasmoid-check patch. It checks if the > interface exists on dbus and is valid. If not, it displays a dialog asking > user what to do (see screenshot). It can also remember the preference. Please > also check spelling/grammar of the text. > > > Diffs > ----- > > main-widget.h 313f74f > main-widget.cpp 28c2d9a > > Diff: http://git.reviewboard.kde.org/r/101918/diff > > > Testing > ------- > > Tested both with plasmoid active and not active, works as expected. > > > Screenshots > ----------- > > The quit dialog > http://git.reviewboard.kde.org/r/101918/s/194/ > > > Thanks, > > Martin > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
