Review Request: Add navigation through history of sent messages in message editor

Dan Vratil dan at progdan.cz
Mon Aug 27 22:19:12 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106245/
-----------------------------------------------------------

Review request for Telepathy.


Description
-------

This implements navigation through history of sent messages by pressing Up and Down arrow keys like in Konversation. It's actually based on code from Konversation, I just didn't implement navigating by scrolling mouse wheel in the message editor as it's possible to have scrollbar in the message box (unlike in Konversation) and it would be extremely annoying for users if scrolling "beyond" end of the text would change the content of the message box.

I also tried populating the history of newly opened chat by a few last lines from last log, but I'm unsure about this feature. Subjectively it felt a bit confusing when I opened a new window and pressing Up key was pasting some text that appeared from "nowhere" and was not related to this session/conversation. But feel free to discuss, I can provide the code if you want to try. Alternatively we can just wait if users will ask for this functionality :)


This addresses bug 303648.
    http://bugs.kde.org/show_bug.cgi?id=303648


Diffs
-----

  lib/chat-text-edit.h 3e2dec0 
  lib/chat-text-edit.cpp e4ca62c 

Diff: http://git.reviewboard.kde.org/r/106245/diff/


Testing
-------


Thanks,

Dan Vratil

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120827/ba7c504e/attachment-0001.html>


More information about the KDE-Telepathy mailing list