Review Request 128867: [KTp/Message] Direction of received message now depends on sender

Alexandr Akulich akulichalexander at gmail.com
Tue Sep 20 11:05:03 UTC 2016


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

(Updated Sept. 20, 2016, 7:05 a.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy.


Changes
-------

Submitted with commit 431db458de1417764ecdef00974fca4c412e19f8 by Alexandr Akulich to branch master.


Repository: ktp-common-internals


Description
-------

If the sender is selfContact, then the direction should be LocalToRemote even for ReceivedMessages.

With this fix Scrollback feature in ktp-text-ui starts to partially work.

Remaining issues (with scrollback):
- If ktp-proxy is on, then sender of received message forced to the channel target, so all received messages would be "incoming";
- Messages in TextUi appears in order of receiving. This means that if scrollback received from the newer to older messages, then the older message would be showed at the end (like if it was been just sent).


Diffs
-----

  KTp/message.cpp 566241e 

Diff: https://git.reviewboard.kde.org/r/128867/diff/


Testing
-------

Compiles, runs. Fixes scrollback for 40% (30% and 30% are broken ktp-proxy and buggy ktp-text-ui, which shows messages in straight instead of chronological message order).


Thanks,

Alexandr Akulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20160920/4af4a3a5/attachment.html>


More information about the KDE-Telepathy mailing list