> On Oct. 14, 2011, 5:38 p.m., David Edmundson wrote: > > dialogs/custom-presence-dialog.cpp, line 80 > > <http://git.reviewboard.kde.org/r/102865/diff/1/?file=38880#file38880line80> > > > > Definitely do not need this and your event filter. > > > > So why does this not work and your event filter does?
Trapping the return key results only in stopping the propagation of that event further, that means that when you press Enter, the dialog won't close. It's not meant to replace the event filter, so..that's why this does not work ;) > On Oct. 14, 2011, 5:38 p.m., David Edmundson wrote: > > dialogs/custom-presence-dialog.cpp, line 149 > > <http://git.reviewboard.kde.org/r/102865/diff/1/?file=38880#file38880line149> > > > > worth checking keyEvent->modifiers() is empty too. Added. Although the docs have a warning: "This function cannot always be trusted. The user can confuse it by pressing both Shift keys simultaneously and releasing one of them, for example." - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102865/#review7341 ----------------------------------------------------------- On Oct. 14, 2011, 5:11 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102865/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2011, 5:11 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Adds event filter that catches the return key and does the needed magic that > is otherwise broken, fixes bug 283802. > > > This addresses bug 283802. > http://bugs.kde.org/show_bug.cgi?id=283802 > > > Diffs > ----- > > dialogs/custom-presence-dialog.h 4a764a9 > dialogs/custom-presence-dialog.cpp 94e2f75 > > Diff: http://git.reviewboard.kde.org/r/102865/diff/diff > > > Testing > ------- > > > Thanks, > > Martin Klapetek > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
