Review Request 113354: Show only one active chat [2/2]

Siddhartha Sahu sh.siddhartha at gmail.com
Thu Oct 24 13:22:33 UTC 2013


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

(Updated Oct. 24, 2013, 6:52 p.m.)


Review request for Telepathy.


Changes
-------

Updating active chat index to Q_PROPERTY.

I need help in the case when a chat is initiated by a contact. Using textChannel->isRequested() fails in that case, and the chat never becomes an active chat. We will want to switch only when user clicks on the contact name, so how to get/store this information?

I considered calling a function which sets the active chat index on the click action. In that case, how to map the contact with the text channel?


Repository: ktp-common-internals


Description
-------

Moves/appends the currently requested chat to the end of the list of chats, which is considered the active chat.
One other way to do this would be to maintain a index variable which carries the active chat index. But I could not find a way to update the model once that index is changed.


Diffs (updated)
-----

  KTp/Declarative/conversations-model.h 1691238 
  KTp/Declarative/conversations-model.cpp 630d403 

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


Testing
-------

Works


Thanks,

Siddhartha Sahu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20131024/2be2de9e/attachment.html>


More information about the KDE-Telepathy mailing list