Review Request: Use AccountsComboBox in AddContactDialog and JoinChatRoomDialog
David Edmundson
david at davidedmundson.co.uk
Wed Nov 14 05:50:41 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107321/
-----------------------------------------------------------
Review request for Telepathy.
Description
-------
Use AccountsComboBox in AddContactDialog and JoinChatRoomDialog
This removes incorrect usages of ContactModel to create lists of accounts in common internals,
as well as removing some duplicate code.
This patch also fixes the JoinChatRoomDialog not updating as accounts become online/offline.
Note this breaks the API for all people using AddContactDialog (ktp-contact-list and ktp-presence-applet)
(for the last time!)
Diffs
-----
KTp/Widgets/add-contact-dialog.h cd4b66e9aabd045be6bf2185f6d262de4639e9eb
KTp/Widgets/add-contact-dialog.cpp ddc26bdaa1e2f95f5cbd28633abaa95f5a0b8738
KTp/Widgets/add-contact-dialog.ui 616fc71782e2721ca78e15df46778b93a7717432
KTp/Widgets/join-chat-room-dialog.cpp 34905396d2d3c9795d1c1b81fdd4725e6d1b88ff
KTp/Widgets/join-chat-room-dialog.ui 2d077c46a42b7f922d497bc3ba07efe421b542de
Diff: http://git.reviewboard.kde.org/r/107321/diff/
Testing
-------
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20121114/60bf03b2/attachment.html>
More information about the KDE-Telepathy
mailing list