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. 24, 2013, 9:13 p.m.)


Review request for Telepathy.


Changes
-------

* Fix loading active chats
* Move index variable to ConversationsModelPrivate


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.cpp 630d403 
  KTp/Declarative/conversations-model.h 1691238 

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/a2d60d84/attachment.html>