----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104551/#review12351 -----------------------------------------------------------
That surely can't be the best way to hide a status notifier. Check out how else it's used in KDE: http://lxr.kde.org/ident?i=KStatusNotifierItem Also your patch will hide all contact notifications if any account goes offline, not just ones that are relevant. Ideally you should be removing the _relevant_ ones from the list m_pendingContacts and then call updateMenus(). Doing this may implicitly solve your hiding the notifier issue too. - David Edmundson On April 11, 2012, 7:01 p.m., Alexander Moses wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104551/ > ----------------------------------------------------------- > > (Updated April 11, 2012, 7:01 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Hi > > I'm working on https://bugs.kde.org/show_bug.cgi?id=294478 . What I've done > is set the SNI to Passive and called setIconByName with an empty QString. But > I don't think this will be sufficient. Please provide your suggestions. > > > Diffs > ----- > > contact-request-handler.cpp 37d9996 > > Diff: http://git.reviewboard.kde.org/r/104551/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Moses > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
