----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/768/#review1243 -----------------------------------------------------------
Hm, looks like I've mislead you (and you didn't test the patch :P ) The keysequence has to be taken from action collection from KopeteChatWindow so you have to set it in KopeteChatWindow::setActiveView So in setActiveView you should add something like m_activeView->editPart()->textEdit()->setSendKeySequence( theSequence ); Also we may in KopeteChatWindow check if the send action has changed (the key sequence) with something like connect( sendAction, SIGNAL(changed()), this, SLOT(updateSendKeySequence()) ); and in KopeteChatWindow::updateSendKeySequence call the m_activeView->editPart()->textEdit()->setSendKeySequence too - Roman On 2009-05-31 08:13:12, Welsh_Dwarf wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/768/ > ----------------------------------------------------------- > > (Updated 2009-05-31 08:13:12) > > > Review request for Kopete. > > > Summary > ------- > > This patch fixes bug 91409: an empty line is sometimes inserted at the start > of a message if the type holds down the enter key for too long. > > > This addresses bug 91409. > https://bugs.kde.org/show_bug.cgi?id=91409 > > > Diffs > ----- > > /trunk/KDE/kdenetwork/kopete/kopete/chatwindow/kopeterichtextwidget.h > 974406 > /trunk/KDE/kdenetwork/kopete/kopete/chatwindow/kopeterichtextwidget.cpp > 974406 > > Diff: http://reviewboard.kde.org/r/768/diff > > > Testing > ------- > > I've sent test messages using a patched kopete and held down, and repeatedly > pressed the enter key to insure that it is no longer possible to insert a > blank line at the start of a message. This fixes the bug. > > A possible complication would be if the user re-configured send to another > key and tried to insert a blank line at the start of the message (to do some > ascii art or something like that). If such a case arrose, the user would have > to put another caracter in fromt (a space or some introductory text) in order > to be able to use new lines. > > > Thanks, > > Welsh_Dwarf > > _______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel