> On Feb. 28, 2013, 5:26 p.m., David Edmundson wrote: > > lib/chat-text-edit.cpp, line 111 > > <http://git.reviewboard.kde.org/r/109202/diff/2/?file=116240#file116240line111> > > > > This won't work if you set the send message shortcut to shift + enter. > > > > e->modifiers() is now non zero, so we would skip sending the message. > > > > I think we want e->matches(m_sendMessageAction->shortcut()); > > David Edmundson wrote: > edit: what I said won't work. > but you do need to check the e->key and e->modifiers in the event and > shortcut are the same.
I finally found out how do we do that. Hell of a job, but it works. - Roman ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109202/#review28275 ----------------------------------------------------------- On March 1, 2013, 9:23 a.m., Roman Nazarenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109202/ > ----------------------------------------------------------- > > (Updated March 1, 2013, 9:23 a.m.) > > > Review request for Telepathy. > > > Description > ------- > > The patch is a mess. QTextEdit handles Key_Return, we need to save Key_Return > from passing to QTextEdit, but we need to know KAction shortcut. And that > shortcut is stored in ChatWindow, which is located far away in the QObject > parenting tree. > This patch allows user to configure custom shortcut, but standard shortcut > (Key_Return) is still working. I have no idea how to disable it without types > casting hell. > > > This addresses bug https://bugs.kde.org/show_bug.cgi?id=308894. > > http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=308894 > > > Diffs > ----- > > app/chat-window.h 018e91a > app/chat-window.cpp c79e3aa > lib/chat-text-edit.h 4279ce5 > lib/chat-text-edit.cpp 19fdb38 > > Diff: http://git.reviewboard.kde.org/r/109202/diff/ > > > Testing > ------- > > > Thanks, > > Roman Nazarenko > >
_______________________________________________ KDE-Telepathy mailing list KDE-Telepathy@kde.org https://mail.kde.org/mailman/listinfo/kde-telepathy
