----------------------------------------------------------- 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. Changes ------- Shortcut handling is now only doing in text area object. Handling was also moved to event() 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 (updated) ----- 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 [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
