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

[Siddhartha Sahu]

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

(Updated Oct. 22, 2013, 2:16 a.m.)


Review request for Telepathy.


Changes
-------

Method 2. I found a way to refresh the model after id change, and now seems to work properly.


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/20131021/49bad749/attachment.html>